Skip to content

RDKBACCL-1524:Intermittently FW upgrade is not working#6

Closed
mahanteshchitt wants to merge 5 commits intodevelopfrom
fix-uncompress-issue
Closed

RDKBACCL-1524:Intermittently FW upgrade is not working#6
mahanteshchitt wants to merge 5 commits intodevelopfrom
fix-uncompress-issue

Conversation

@mahanteshchitt
Copy link
Copy Markdown
Contributor

Reason For Change: Intermittently Firmware upgrade is failing showing the error uncompression failed.
Test Procedure : Firmware upgrade should work as expected.
Risks: None.

Reason For Change: Intermittently Firmware upgrade is failing showing the error uncompression failed.
Test Procedure : Firmware upgrade should work as expected.
Risks: None.

Signed-off-by: mahanteshchitt <mahantesh_chittapur@comcast.com>
Comment thread source/fwupgrade/fwupgrade_hal.c Outdated
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please replace all these system API call with run_command.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

okay, will address.

return -1;
}

ret = run_command(cmd, output, sizeof(output));

Check failure

Code scanning / CodeQL

Uncontrolled data used in OS command Critical

This argument to an OS command is derived from
user input (string read by fgets)
, dangerously concatenated into
snprintf output argument
, and then passed to run_command(cmd), which calls popen(__command).
This argument to an OS command is derived from
user input (string read by fgets)
, dangerously concatenated into
snprintf output argument
, and then passed to run_command(cmd), which calls popen(__command).
This argument to an OS command is derived from
user input (string read by fgets)
, dangerously concatenated into
snprintf output argument
, and then passed to run_command(cmd), which calls popen(__command).
@github-actions github-actions Bot locked and limited conversation to collaborators Mar 10, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants