|
 |
f239de |
From 8a97e4f67f75a4584f7562b7e5d866431c88152e Mon Sep 17 00:00:00 2001
|
|
|
f239de |
From: Lukas Czerner <lczerner@redhat.com>
|
|
|
f239de |
Date: Fri, 5 Jun 2020 10:14:40 +0200
|
|
|
f239de |
Subject: [PATCH 09/46] e2fsck: use size_t instead of int in string_copy()
|
|
|
f239de |
Content-Type: text/plain
|
|
|
f239de |
|
|
|
f239de |
len argument in string_copy() is int, but it is used with malloc(),
|
|
|
f239de |
strlen(), strncpy() and some callers use sizeof() to pass value in. So
|
|
|
f239de |
it really ought to be size_t rather than int. Fix it.
|
|
|
f239de |
|
|
|
f239de |
Signed-off-by: Lukas Czerner <lczerner@redhat.com>
|
|
|
f239de |
Reviewed-by: Andreas Dilger <adilger@dilger.ca>
|
|
|
f239de |
Signed-off-by: Theodore Ts'o <tytso@mit.edu>
|
|
|
f239de |
---
|
|
|
f239de |
e2fsck/e2fsck.h | 2 +-
|
|
|
f239de |
e2fsck/util.c | 2 +-
|
|
|
f239de |
2 files changed, 2 insertions(+), 2 deletions(-)
|
|
|
f239de |
|
|
|
f239de |
diff --git a/e2fsck/e2fsck.h b/e2fsck/e2fsck.h
|
|
|
f239de |
index feb605c7..7e0895c2 100644
|
|
|
f239de |
--- a/e2fsck/e2fsck.h
|
|
|
f239de |
+++ b/e2fsck/e2fsck.h
|
|
|
f239de |
@@ -608,7 +608,7 @@ extern void log_err(e2fsck_t ctx, const char *fmt, ...)
|
|
|
f239de |
extern void e2fsck_read_bitmaps(e2fsck_t ctx);
|
|
|
f239de |
extern void e2fsck_write_bitmaps(e2fsck_t ctx);
|
|
|
f239de |
extern void preenhalt(e2fsck_t ctx);
|
|
|
f239de |
-extern char *string_copy(e2fsck_t ctx, const char *str, int len);
|
|
|
f239de |
+extern char *string_copy(e2fsck_t ctx, const char *str, size_t len);
|
|
|
f239de |
extern int fs_proc_check(const char *fs_name);
|
|
|
f239de |
extern int check_for_modules(const char *fs_name);
|
|
|
f239de |
#ifdef RESOURCE_TRACK
|
|
|
f239de |
diff --git a/e2fsck/util.c b/e2fsck/util.c
|
|
|
f239de |
index d98b8e47..88e0ea8a 100644
|
|
|
f239de |
--- a/e2fsck/util.c
|
|
|
f239de |
+++ b/e2fsck/util.c
|
|
|
f239de |
@@ -135,7 +135,7 @@ void *e2fsck_allocate_memory(e2fsck_t ctx, unsigned long size,
|
|
|
f239de |
}
|
|
|
f239de |
|
|
|
f239de |
char *string_copy(e2fsck_t ctx EXT2FS_ATTR((unused)),
|
|
|
f239de |
- const char *str, int len)
|
|
|
f239de |
+ const char *str, size_t len)
|
|
|
f239de |
{
|
|
|
f239de |
char *ret;
|
|
|
f239de |
|
|
|
f239de |
--
|
|
|
f239de |
2.35.1
|
|
|
f239de |
|