[git commit] fix unsafe bb_perror_msg(filename) calls

Denys Vlasenko vda.linux at googlemail.com
Thu Sep 27 13:35:10 UTC 2012


commit: http://git.busybox.net/busybox/commit/?id=9371043698933452bb69566f279a6e40af961c8c
branch: http://git.busybox.net/busybox/commit/?id=refs/heads/master

Signed-off-by: Denys Vlasenko <vda.linux at googlemail.com>
---
 coreutils/stat.c                 |    4 ++--
 e2fsprogs/old_e2fsprogs/lsattr.c |    2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/coreutils/stat.c b/coreutils/stat.c
index 3fb212f..e38c8f6 100644
--- a/coreutils/stat.c
+++ b/coreutils/stat.c
@@ -442,7 +442,7 @@ static bool do_statfs(const char *filename, const char *format)
 		     : getfilecon(filename, &scontext)
 		    ) < 0
 		) {
-			bb_perror_msg(filename);
+			bb_simple_perror_msg(filename);
 			return 0;
 		}
 	}
@@ -555,7 +555,7 @@ static bool do_stat(const char *filename, const char *format)
 		     : getfilecon(filename, &scontext)
 		    ) < 0
 		) {
-			bb_perror_msg(filename);
+			bb_simple_perror_msg(filename);
 			return 0;
 		}
 	}
diff --git a/e2fsprogs/old_e2fsprogs/lsattr.c b/e2fsprogs/old_e2fsprogs/lsattr.c
index 9e0e4cb..9eab68b 100644
--- a/e2fsprogs/old_e2fsprogs/lsattr.c
+++ b/e2fsprogs/old_e2fsprogs/lsattr.c
@@ -93,7 +93,7 @@ static int lsattr_dir_proc(const char *dir_name, struct dirent *de,
 	path = concat_path_file(dir_name, de->d_name);
 
 	if (lstat(path, &st) == -1)
-		bb_perror_msg(path);
+		bb_simple_perror_msg(path);
 	else {
 		if (de->d_name[0] != '.' || (flags & OPT_ALL)) {
 			list_attributes(path);


More information about the busybox-cvs mailing list