To ensure that the firmware tag is properly null-terminated. When copying firmware tag strings to prevent buffer overflows and ensure data integrity.
changes has been made in 3 positions: 1: update firmware tag 2: backend update firmware tag 3: backend update firmware tag with unipro
Signed-off-by: Rujra Bhatt braker.noob.kernel@gmail.com --- drivers/staging/greybus/Documentation/firmware/firmware.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/greybus/Documentation/firmware/firmware.c b/drivers/staging/greybus/Documentation/firmware/firmware.c index 765d69faa..f37904b91 100644 --- a/drivers/staging/greybus/Documentation/firmware/firmware.c +++ b/drivers/staging/greybus/Documentation/firmware/firmware.c @@ -63,7 +63,7 @@ static int update_intf_firmware(int fd) intf_load.major = 0; intf_load.minor = 0;
- strncpy((char *)&intf_load.firmware_tag, firmware_tag, + strscpy((char *)&intf_load.firmware_tag, firmware_tag, GB_FIRMWARE_U_TAG_MAX_SIZE);
ret = ioctl(fd, FW_MGMT_IOC_INTF_LOAD_AND_VALIDATE, &intf_load); @@ -101,7 +101,7 @@ static int update_backend_firmware(int fd) /* Get Backend Firmware Version */ printf("Getting Backend Firmware Version\n");
- strncpy((char *)&backend_fw_info.firmware_tag, firmware_tag, + strscpy((char *)&backend_fw_info.firmware_tag, firmware_tag, GB_FIRMWARE_U_TAG_MAX_SIZE);
retry_fw_version: @@ -129,7 +129,7 @@ static int update_backend_firmware(int fd) /* Try Backend Firmware Update over Unipro */ printf("Updating Backend Firmware\n");
- strncpy((char *)&backend_update.firmware_tag, firmware_tag, + strscpy((char *)&backend_update.firmware_tag, firmware_tag, GB_FIRMWARE_U_TAG_MAX_SIZE);
retry_fw_update: -- 2.43.0
On Wed, May 21, 2025 at 10:26:52AM +0530, rujra wrote:
To ensure that the firmware tag is properly null-terminated. When copying firmware tag strings to prevent buffer overflows and ensure data integrity.
changes has been made in 3 positions: 1: update firmware tag 2: backend update firmware tag 3: backend update firmware tag with unipro
Signed-off-by: Rujra Bhatt braker.noob.kernel@gmail.com
Your From: line does not match this :(
Also, why is this a "RESEND"?
thanks,
greg k-h
Hi greg,
why is this a RESEND ?:
I had sent the same patch 4 days ago and didn't get any reply , hence tried to resend the same patch for the same,
here is earlier mail : https://mail.google.com/mail/u/1/?ik=f63b03515e&view=om&permmsgid=ms... ,
line does not match this:
sorry , i could not get it what it is exactly ?, if possible can you share some insights or example so that from in future i would get it right.
thank you, rujra bhatt
On Wed, May 21, 2025 at 11:00 AM Greg KH gregkh@linuxfoundation.org wrote:
On Wed, May 21, 2025 at 10:26:52AM +0530, rujra wrote:
To ensure that the firmware tag is properly null-terminated. When copying firmware tag strings to prevent buffer overflows and ensure data integrity.
changes has been made in 3 positions: 1: update firmware tag 2: backend update firmware tag 3: backend update firmware tag with unipro
Signed-off-by: Rujra Bhatt braker.noob.kernel@gmail.com
Your From: line does not match this :(
Also, why is this a "RESEND"?
thanks,
greg k-h
A: http://en.wikipedia.org/wiki/Top_post Q: Were do I find info about this thing called top-posting? A: Because it messes up the order in which people normally read text. Q: Why is top-posting such a bad thing? A: Top-posting. Q: What is the most annoying thing in e-mail?
A: No. Q: Should I include quotations after my reply?
http://daringfireball.net/2007/07/on_top
On Wed, May 21, 2025 at 11:11:26AM +0530, rujra wrote:
Hi greg,
why is this a RESEND ?:
I had sent the same patch 4 days ago and didn't get any reply , hence tried to resend the same patch for the same,
here is earlier mail : https://mail.google.com/mail/u/1/?ik=f63b03515e&view=om&permmsgid=ms...
That is a link to _your_ account, not a public record of your email :(
,
line does not match this:
sorry , i could not get it what it is exactly ?, if possible can you share some insights or example so that from in future i would get it right.
Your "From:" line in your email does not match with the signed-off-by line.
thanks,
greg k-h