From 5e1b2ca1161cba481ccf4873427389f59dbc23e0 Mon Sep 17 00:00:00 2001 From: John Beppu Date: Fri, 11 Feb 2000 12:52:55 +0000 Subject: [PATCH] + memory allocation/deallocation is less tolerant of evil. --- fsck_minix.c | 23 +++++++++++++++++++++-- util-linux/fsck_minix.c | 23 +++++++++++++++++++++-- 2 files changed, 42 insertions(+), 4 deletions(-) diff --git a/fsck_minix.c b/fsck_minix.c index 9da79748f..cfa973ecf 100644 --- a/fsck_minix.c +++ b/fsck_minix.c @@ -146,7 +146,7 @@ static int termios_set = 0; #define MAX_DEPTH 32 static int name_depth = 0; // static char name_list[MAX_DEPTH][PATH_MAX + 1]; -static char **name_list; +static char **name_list = NULL; static char *inode_buffer = NULL; @@ -1248,14 +1248,33 @@ static void alloc_name_list(void) int i; name_list = malloc(sizeof(char *) * MAX_DEPTH); + if (!name_list) { + fprintf(stderr,"fsck_minix: name_list: %s\n", strerror(errno)); + exit(1); + } for (i = 0; i < MAX_DEPTH; i++) { name_list[i] = malloc(sizeof(char) * PATH_MAX + 1); + if (!name_list[i]) { + fprintf(stderr,"fsck_minix: name_list: %s\n", strerror(errno)); + exit(1); + } } } +/* execute this atexit() to deallocate name_list[] */ +/* piptigger was here */ static void free_name_list(void) { - if (name_list) free(name_list); + int i; + + if (name_list) { + for (i = 0; i < MAX_DEPTH; i++) { + if (name_list[i]) { + free(name_list[i]); + } + } + free(name_list); + } } extern int fsck_minix_main(int argc, char **argv) diff --git a/util-linux/fsck_minix.c b/util-linux/fsck_minix.c index 9da79748f..cfa973ecf 100644 --- a/util-linux/fsck_minix.c +++ b/util-linux/fsck_minix.c @@ -146,7 +146,7 @@ static int termios_set = 0; #define MAX_DEPTH 32 static int name_depth = 0; // static char name_list[MAX_DEPTH][PATH_MAX + 1]; -static char **name_list; +static char **name_list = NULL; static char *inode_buffer = NULL; @@ -1248,14 +1248,33 @@ static void alloc_name_list(void) int i; name_list = malloc(sizeof(char *) * MAX_DEPTH); + if (!name_list) { + fprintf(stderr,"fsck_minix: name_list: %s\n", strerror(errno)); + exit(1); + } for (i = 0; i < MAX_DEPTH; i++) { name_list[i] = malloc(sizeof(char) * PATH_MAX + 1); + if (!name_list[i]) { + fprintf(stderr,"fsck_minix: name_list: %s\n", strerror(errno)); + exit(1); + } } } +/* execute this atexit() to deallocate name_list[] */ +/* piptigger was here */ static void free_name_list(void) { - if (name_list) free(name_list); + int i; + + if (name_list) { + for (i = 0; i < MAX_DEPTH; i++) { + if (name_list[i]) { + free(name_list[i]); + } + } + free(name_list); + } } extern int fsck_minix_main(int argc, char **argv) -- 2.25.1