[vamos-dev] [PATCH 1/3] fix literal error warning

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


Fixed in git:

commit 9371043698933452bb69566f279a6e40af961c8c
Author: Denys Vlasenko <vda.linux at googlemail.com>
Date:   Thu Sep 27 15:35:10 2012 +0200

    fix unsafe bb_perror_msg(filename) calls

    Signed-off-by: Denys Vlasenko <vda.linux at googlemail.com>

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 mailing list