lkml.org 
[lkml]   [2024]   [Jan]   [2]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
Subject[PATCH] block: fix length of strscpy()
Date
In commit 146afeb235ccec10c17ad8ea26327c0c79dbd968 ("block: use strscpy()
to instead of strncpy()") , the length that should now represent the length
of the string with the terminating NULL was not updated alongside the
change.

This has caused blkdevparts= definition on kernel cmdline to be not
correctly recognized and partitions not correctly initialized, breaking any
device relying on such partitions to boot, on stable releases since 6.6

This patch fixes the lengths to contain the terminating NULL.

Cc: stable@vger.kernel.org # 6.6.x
Signed-off-by: Guoxin Pu <pugokushin@gmail.com>
---
block/partitions/cmdline.c | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/block/partitions/cmdline.c b/block/partitions/cmdline.c
index c03bc105e575..c2aac5f4ab82 100644
--- a/block/partitions/cmdline.c
+++ b/block/partitions/cmdline.c
@@ -79,8 +79,8 @@ static int parse_subpart(struct cmdline_subpart **subpart, char *partdef)
goto fail;
}

- length = min_t(int, next - partdef,
- sizeof(new_subpart->name) - 1);
+ length = min_t(int, next - partdef + 1,
+ sizeof(new_subpart->name));
strscpy(new_subpart->name, partdef, length);

partdef = ++next;
@@ -138,7 +138,7 @@ static int parse_parts(struct cmdline_parts **parts, const char *bdevdef)
goto fail;
}

- length = min_t(int, next - bdevdef, sizeof(newparts->name) - 1);
+ length = min_t(int, next - bdevdef + 1, sizeof(newparts->name));
strscpy(newparts->name, bdevdef, length);
newparts->nr_subparts = 0;

@@ -148,8 +148,8 @@ static int parse_parts(struct cmdline_parts **parts, const char *bdevdef)
bdevdef = next;
next = strchr(bdevdef, ',');

- length = (!next) ? (sizeof(buf) - 1) :
- min_t(int, next - bdevdef, sizeof(buf) - 1);
+ length = (!next) ? sizeof(buf) :
+ min_t(int, next - bdevdef + 1, sizeof(buf));

strscpy(buf, bdevdef, length);

@@ -262,7 +262,7 @@ static int add_part(int slot, struct cmdline_subpart *subpart,

info = &state->parts[slot].info;

- label_min = min_t(int, sizeof(info->volname) - 1,
+ label_min = min_t(int, sizeof(info->volname),
sizeof(subpart->name));
strscpy(info->volname, subpart->name, label_min);

--
2.43.0

\
 
 \ /
  Last update: 2024-01-01 18:54    [W:0.071 / U:0.644 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site