RDKEMW-6891:Coverity errors fix for xdial#166
Merged
Conversation
arun-madhavan-013
requested changes
Aug 27, 2025
arun-madhavan-013
requested changes
Sep 2, 2025
Fixed review comments
arun-madhavan-013
approved these changes
Sep 3, 2025
apatel859
approved these changes
Sep 3, 2025
apatel859
added a commit
that referenced
this pull request
Sep 25, 2025
* RDKECMF-213 Add CODEOWNERS file * RDKEMW-254: libsoup3 support Signed-off-by: apatel859 <amit_patel5@comcast.com> * RDKTV-35185: Add sync between ssdp_http_server_callback and gdial_ssdp_set_friendlyname Signed-off-by: yuvaramachandran_gurusamy <yuvaramachandran_gurusamy@comcast.com> * RDKEMW-2033: Fix coverity related issue Reason for change: Fix issues identified within xcast Test Procedure: Risks: low Priority: P1 Signed-off-by:Hayden Gfeller <Hayden_Gfeller@comcast.com> * RDK-55044: [RDKE] Implement DIAL requirement to use on EU product Reason for change: Implemented setManufacturerName and setModelName APIs for DIAL Server name configuration maintained the additional data url to specific app Test Procedure: DIAL should work Risks: None Priority: P1 Signed-off-by: yuvaramachandran_gurusamy <yuvaramachandran_gurusamy@comcast.com> * RDK-56672: Fix parsing issue while restrict addtionalData url per app Signed-off-by: yuvaramachandran_gurusamy <yuvaramachandran_gurusamy@comcast.com> * RDKEMW-2278: Removal of WPEFrameworkSecurity Agent Utility Reason for change: Added DISABLE_SECURITY_TOKEN Flag to disable the WPEFrameworkSecurity Token generation changes Test Procedure: please referred from the ticket Risks: Medium Signed-off-by: Thamim Razith <ThamimRazith_AbbasAli@comcast.com> * RDKEMW-4129: Prepare native build environment Signed-off-by: yuvaramachandran_gurusamy <yuvaramachandran_gurusamy@comcast.com> * RDKEMW-4129: Test Signed-off-by: yuvaramachandran_gurusamy <yuvaramachandran_gurusamy@comcast.com> * RDKEMW-4129: Prepare native build script Signed-off-by: yuvaramachandran_gurusamy <yuvaramachandran_gurusamy@comcast.com> * remmove duplicates from build_xxx script Signed-off-by: apatel859 <amit_patel5@comcast.com> * Update gdial.cpp * RDKEMW-5927: Crash fix during soup_uri_to_string API call (#163) Signed-off-by: yuvaramachandran_gurusamy <yuvaramachandran_gurusamy@comcast.com> * RDKEMW-6891:Coverity errors fix for xdial (#166) * RDKEMW-6891: Coverity errors fix for xdial * Update gdial.cpp Fixed review comments * Deploy cla action --------- Signed-off-by: apatel859 <amit_patel5@comcast.com> Signed-off-by: yuvaramachandran_gurusamy <yuvaramachandran_gurusamy@comcast.com> Signed-off-by: Thamim Razith <ThamimRazith_AbbasAli@comcast.com> Co-authored-by: Simon Chung <simon.chung@rdkcentral.com> Co-authored-by: Stephen Barrett <sbarre01@users.noreply.github.com> Co-authored-by: apatel859 <amit_patel5@comcast.com> Co-authored-by: apatel859 <48992974+apatel859@users.noreply.github.com> Co-authored-by: hgfell683 <107510770+hgfell683@users.noreply.github.com> Co-authored-by: tabbas651 <thamimrazith_abbasali@comcast.com> Co-authored-by: ssitar583 <soundaryaa_sitaram@comcast.com> Co-authored-by: mukesh972 <mukesh.m@tataelxsi.co.in> Co-authored-by: rdkcmf <github@code.rdkcentral.com> Co-authored-by: Alan Ryan <20208488+Alan-Ryan@users.noreply.github.com>
apatel859
added a commit
that referenced
this pull request
Oct 23, 2025
* RDKECMF-213 Add CODEOWNERS file * RDKEMW-254: libsoup3 support * RDKTV-35185: Add sync between ssdp_http_server_callback and gdial_ssdp_set_friendlyname * RDKEMW-2033: Fix coverity related issue * RDK-55044: [RDKE] Implement DIAL requirement to use on EU product * RDK-56672: Fix parsing issue while restrict addtionalData url per app * RDKEMW-2278: Removal of WPEFrameworkSecurity Agent Utility * RDKEMW-4129: Prepare native build environment * RDKEMW-4129: Test * RDKEMW-4129: Prepare native build script * remmove duplicates from build_xxx script * Update gdial.cpp * RDKEMW-5927: Crash fix during soup_uri_to_string API call (#163) * RDKEMW-6891:Coverity errors fix for xdial (#166) * RDKEMW-6891: Coverity errors fix for xdial * Update gdial.cpp Fixed review comments * Deploy cla action --------- Signed-off-by: apatel859 <amit_patel5@comcast.com> Signed-off-by: yuvaramachandran_gurusamy <yuvaramachandran_gurusamy@comcast.com> Signed-off-by: Thamim Razith <ThamimRazith_AbbasAli@comcast.com> Co-authored-by: Simon Chung <simon.chung@rdkcentral.com> Co-authored-by: Stephen Barrett <sbarre01@users.noreply.github.com> Co-authored-by: apatel859 <amit_patel5@comcast.com> Co-authored-by: apatel859 <48992974+apatel859@users.noreply.github.com> Co-authored-by: hgfell683 <107510770+hgfell683@users.noreply.github.com> Co-authored-by: tabbas651 <thamimrazith_abbasali@comcast.com> Co-authored-by: ssitar583 <soundaryaa_sitaram@comcast.com> Co-authored-by: mukesh972 <mukesh.m@tataelxsi.co.in> Co-authored-by: rdkcmf <github@code.rdkcentral.com> Co-authored-by: Alan Ryan <20208488+Alan-Ryan@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.