* [PATCH] Rename progname as it is provided by libc @ 2017-09-02 21:54 Khem Raj 2017-09-02 22:20 ` Dave Chinner 2017-09-08 16:25 ` Eric Sandeen 0 siblings, 2 replies; 8+ messages in thread From: Khem Raj @ 2017-09-02 21:54 UTC (permalink / raw) To: linux-xfs; +Cc: Khem Raj Rename local variable progname to avoid a clash with libc global symbols Signed-off-by: Khem Raj <raj.khem@gmail.com> --- io/init.c | 10 +++++----- mdrestore/xfs_mdrestore.c | 10 +++++----- quota/init.c | 10 +++++----- spaceman/init.c | 8 ++++---- 4 files changed, 19 insertions(+), 19 deletions(-) diff --git a/io/init.c b/io/init.c index 20d5f80..e82e101 100644 --- a/io/init.c +++ b/io/init.c @@ -23,7 +23,7 @@ #include "init.h" #include "io.h" -char *progname; +char *io_progname; int exitcode; int expert; int idlethread; @@ -35,7 +35,7 @@ usage(void) { fprintf(stderr, _("Usage: %s [-adfinrRstVx] [-m mode] [-p prog] [[-c|-C] cmd]... file\n"), - progname); + io_progname); exit(1); } @@ -142,7 +142,7 @@ init( xfs_fsop_geom_t geometry = { 0 }; struct fs_path fsp; - progname = basename(argv[0]); + io_progname = basename(argv[0]); setlocale(LC_ALL, ""); bindtextdomain(PACKAGE, LOCALEDIR); textdomain(PACKAGE); @@ -186,7 +186,7 @@ init( flags |= IO_NONBLOCK; break; case 'p': - progname = optarg; + io_progname = optarg; break; case 'r': flags |= IO_READONLY; @@ -207,7 +207,7 @@ init( expert = 1; break; case 'V': - printf(_("%s version %s\n"), progname, VERSION); + printf(_("%s version %s\n"), io_progname, VERSION); exit(0); default: usage(); diff --git a/mdrestore/xfs_mdrestore.c b/mdrestore/xfs_mdrestore.c index 9d1b4e8..b840a54 100644 --- a/mdrestore/xfs_mdrestore.c +++ b/mdrestore/xfs_mdrestore.c @@ -19,7 +19,7 @@ #include "libxfs.h" #include "xfs_metadump.h" -char *progname; +char *mdrestore_progname; int show_progress = 0; int show_info = 0; int progress_since_warning = 0; @@ -30,7 +30,7 @@ fatal(const char *msg, ...) va_list args; va_start(args, msg); - fprintf(stderr, "%s: ", progname); + fprintf(stderr, "%s: ", mdrestore_progname); vfprintf(stderr, msg, args); exit(1); } @@ -194,7 +194,7 @@ perform_restore( static void usage(void) { - fprintf(stderr, "Usage: %s [-V] [-g] source target\n", progname); + fprintf(stderr, "Usage: %s [-V] [-g] source target\n", mdrestore_progname); exit(1); } @@ -212,7 +212,7 @@ main( struct stat statbuf; int is_target_file; - progname = basename(argv[0]); + mdrestore_progname = basename(argv[0]); while ((c = getopt(argc, argv, "giV")) != EOF) { switch (c) { @@ -223,7 +223,7 @@ main( show_info = 1; break; case 'V': - printf("%s version %s\n", progname, VERSION); + printf("%s version %s\n", mdrestore_progname, VERSION); exit(0); default: usage(); diff --git a/quota/init.c b/quota/init.c index d45dc4c..46403de 100644 --- a/quota/init.c +++ b/quota/init.c @@ -21,7 +21,7 @@ #include "input.h" #include "init.h" -char *progname; +char *quota_progname; int exitcode; int expert; bool foreign_allowed = false; @@ -47,7 +47,7 @@ usage(void) { fprintf(stderr, _("Usage: %s [-V] [-x] [-f] [-p prog] [-c cmd]... [-d project]... [path]\n"), - progname); + quota_progname); exit(1); } @@ -147,7 +147,7 @@ init( { int c; - progname = basename(argv[0]); + quota_progname = basename(argv[0]); setlocale(LC_ALL, ""); bindtextdomain(PACKAGE, LOCALEDIR); textdomain(PACKAGE); @@ -173,13 +173,13 @@ init( projid_file = optarg; break; case 'p': - progname = optarg; + quota_progname = optarg; break; case 'x': expert++; break; case 'V': - printf(_("%s version %s\n"), progname, VERSION); + printf(_("%s version %s\n"), quota_progname, VERSION); exit(0); default: usage(); diff --git a/spaceman/init.c b/spaceman/init.c index b3eface..bedf112 100644 --- a/spaceman/init.c +++ b/spaceman/init.c @@ -23,7 +23,7 @@ #include "path.h" #include "space.h" -char *progname; +char *spaceman_progname; int exitcode; void @@ -31,7 +31,7 @@ usage(void) { fprintf(stderr, _("Usage: %s [-c cmd] file\n"), - progname); + spaceman_progname); exit(1); } @@ -74,7 +74,7 @@ init( xfs_fsop_geom_t geometry = { 0 }; struct fs_path fsp; - progname = basename(argv[0]); + spaceman_progname = basename(argv[0]); setlocale(LC_ALL, ""); bindtextdomain(PACKAGE, LOCALEDIR); textdomain(PACKAGE); @@ -86,7 +86,7 @@ init( add_user_command(optarg); break; case 'V': - printf(_("%s version %s\n"), progname, VERSION); + printf(_("%s version %s\n"), spaceman_progname, VERSION); exit(0); default: usage(); -- 2.14.1 ^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH] Rename progname as it is provided by libc 2017-09-02 21:54 [PATCH] Rename progname as it is provided by libc Khem Raj @ 2017-09-02 22:20 ` Dave Chinner 2017-09-02 23:42 ` Khem Raj ` (2 more replies) 2017-09-08 16:25 ` Eric Sandeen 1 sibling, 3 replies; 8+ messages in thread From: Dave Chinner @ 2017-09-02 22:20 UTC (permalink / raw) To: Khem Raj; +Cc: linux-xfs On Sat, Sep 02, 2017 at 02:54:51PM -0700, Khem Raj wrote: > Rename local variable progname to avoid a clash with libc > global symbols What libc is causing this problem? And why is it considered ok for that libc to pollute the global variable namespace with it's own internal variables? Cheers, Dave. -- Dave Chinner david@fromorbit.com ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] Rename progname as it is provided by libc 2017-09-02 22:20 ` Dave Chinner @ 2017-09-02 23:42 ` Khem Raj 2017-09-03 7:21 ` Khem Raj 2017-09-03 7:48 ` Christoph Hellwig 2 siblings, 0 replies; 8+ messages in thread From: Khem Raj @ 2017-09-02 23:42 UTC (permalink / raw) To: Dave Chinner; +Cc: linux-xfs On Sat, Sep 2, 2017 at 3:20 PM, Dave Chinner <david@fromorbit.com> wrote: > On Sat, Sep 02, 2017 at 02:54:51PM -0700, Khem Raj wrote: >> Rename local variable progname to avoid a clash with libc >> global symbols > > What libc is causing this problem? > > And why is it considered ok for that libc to pollute the global > variable namespace with it's own internal variables? > I am building with -staitc -pie options xfs_mdrestore.o: relocation R_ARM_REL32 against external or undefined symbol `progname' can not be used when making a PIE executable; recompile with -fPIC libcs have __progname, So I will have to check where is progname coming from probably an external header. I think renaming to avoid conflict is good thing regardless. > Cheers, > > Dave. > -- > Dave Chinner > david@fromorbit.com ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] Rename progname as it is provided by libc 2017-09-02 22:20 ` Dave Chinner 2017-09-02 23:42 ` Khem Raj @ 2017-09-03 7:21 ` Khem Raj 2017-09-03 7:48 ` Christoph Hellwig 2 siblings, 0 replies; 8+ messages in thread From: Khem Raj @ 2017-09-03 7:21 UTC (permalink / raw) To: Dave Chinner; +Cc: linux-xfs On Sun, Sep 03, 2017 at 08:20:03AM +1000, Dave Chinner wrote: > On Sat, Sep 02, 2017 at 02:54:51PM -0700, Khem Raj wrote: > > Rename local variable progname to avoid a clash with libc > > global symbols > > What libc is causing this problem? > > And why is it considered ok for that libc to pollute the global > variable namespace with it's own internal variables? A closer look reveals that its getting the function from another local library which is used on the linker commandline. below is full cmdline ../arm-bec-linux-musleabi-libtool --quiet --tag=CC --mode=link ../../recipe-sysroot-native/usr/bin/arm-bec-linux-musleabi/arm-bec-linux-musleabi-clang -march=armv7ve -mthumb -mfpu=neon-vfpv4 -mfloat-abi=hard -mcpu=cortex-a7 -mlittle-endian -D__extern_always_inline=inline -no-integrated-as -Wno-error=unused-command-line-argument -Qunused-arguments -fstack-protector-strong -D_FORTIFY_SOURCE=2 -Wformat -Wformat-security -Werror=format-security --sysroot=/mnt/a/oe/build/tmp/work/cortexa7t2hf-neon-vfpv4-bec-linux-musleabi/xfsprogs/4.12.0-r0/recipe-sysroot -o xfs_io -Wl,-O1 -Wl,--hash-style=gnu -Wl,--as-needed -fstack-protector-strong -Wl,-z,relro,-z,now -Wl,-O1 -Wl,--hash-style=gnu -Wl,--as-needed -fstack-protector-strong -Wl,-z,relro,-z,now -Wl,-O1 -Wl,--hash-style=gnu -Wl,--as-needed -fstack-protector-strong -Wl,-z,relro,-z,now -Wl,-O1 -Wl,--hash-style=gnu -Wl,--as-needed -fstack-protector-strong -Wl,-z,relro,-z,now -static-libtool-libs init.o attr.o bmap.o cowextsize.o encrypt.o file.o freeze.o fsync.o getrusage.o imap.o link.o mmap.o open.o parent.o pread.o prealloc.o pwrite.o reflink.o seek.o shutdown.o stat.o sync.o truncate.o utimes.o fadvise.o madvise.o mincore.o sendfile.o fiemap.o inject.o resblks.o copy_file_range.o sync_file_range.o readdir.o fsmap.o ../libxcmd/libxcmd.la ../libhandle/libhandle.la -luuid -lpthread -lreadline libxfs.so.0 init.o (progname) and also from init.o object from current directory $ readelf -sW init.o | grep progname (09-03 00:17) 56: 00000004 4 OBJECT GLOBAL DEFAULT COM progname readelf -sW ../libxfs/.libs/libxfs.so.0 | grep progname (09-03 00:19) 214: 0005d004 4 OBJECT GLOBAL DEFAULT 18 progname Thats what is causing the conflict. > > Cheers, > > Dave. > -- > Dave Chinner > david@fromorbit.com ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] Rename progname as it is provided by libc 2017-09-02 22:20 ` Dave Chinner 2017-09-02 23:42 ` Khem Raj 2017-09-03 7:21 ` Khem Raj @ 2017-09-03 7:48 ` Christoph Hellwig 2 siblings, 0 replies; 8+ messages in thread From: Christoph Hellwig @ 2017-09-03 7:48 UTC (permalink / raw) To: Dave Chinner; +Cc: Khem Raj, linux-xfs On Sun, Sep 03, 2017 at 08:20:03AM +1000, Dave Chinner wrote: > On Sat, Sep 02, 2017 at 02:54:51PM -0700, Khem Raj wrote: > > Rename local variable progname to avoid a clash with libc > > global symbols > > What libc is causing this problem? > > And why is it considered ok for that libc to pollute the global > variable namespace with it's own internal variables? I don't think it's libc. It's our own little mess where we want to set a global progname for libxfs use in the tools. The right interface is to have a private libxfs_progname routine that isn't accessible by the tools.. ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] Rename progname as it is provided by libc 2017-09-02 21:54 [PATCH] Rename progname as it is provided by libc Khem Raj 2017-09-02 22:20 ` Dave Chinner @ 2017-09-08 16:25 ` Eric Sandeen 2017-09-08 16:29 ` Khem Raj 2017-09-08 18:13 ` Khem Raj 1 sibling, 2 replies; 8+ messages in thread From: Eric Sandeen @ 2017-09-08 16:25 UTC (permalink / raw) To: Khem Raj, linux-xfs On 9/2/17 4:54 PM, Khem Raj wrote: > Rename local variable progname to avoid a clash with libc > global symbols > > Signed-off-by: Khem Raj <raj.khem@gmail.com> > --- > io/init.c | 10 +++++----- > mdrestore/xfs_mdrestore.c | 10 +++++----- > quota/init.c | 10 +++++----- > spaceman/init.c | 8 ++++---- > 4 files changed, 19 insertions(+), 19 deletions(-) > First of all, this doesn't build. Please don't send untested patches... [LD] xfs_io attr.o: In function `chattr_f': /mnt/test2/git/xfsprogs-maint/io/attr.c:316: undefined reference to `progname' /mnt/test2/git/xfsprogs-maint/io/attr.c:310: undefined reference to `progname' /mnt/test2/git/xfsprogs-maint/io/attr.c:332: undefined reference to `progname' /mnt/test2/git/xfsprogs-maint/io/attr.c:326: undefined reference to `progname' attr.o: In function `chattr_callback': /mnt/test2/git/xfsprogs-maint/io/attr.c:252: undefined reference to `progname' attr.o:/mnt/test2/git/xfsprogs-maint/io/attr.c:255: more undefined references to `progname' follow collect2: ld returned 1 exit status gmake[3]: *** [xfs_io] Error 1 gmake[2]: *** [io] Error 2 make[1]: *** [default] Error 2 make: *** [default] Error 2 second: is progname really a global symbol? I know of __progname, but not progname. i.e. this works: #include <stdio.h> extern const char *__progname; int main( int argc, char **argv) { printf("progname: %s\n", __progname); } # gcc -o test test.c # ./test progname: test but this doesn't: #include <stdio.h> extern const char *progname; int main( int argc, char **argv) { printf("progname %s\n", progname); } # gcc -o test test.c /tmp/ccjaZrXZ.o: In function `main': test.c:(.text+0x12): undefined reference to `progname' collect2: ld returned 1 exit status -Eric > diff --git a/io/init.c b/io/init.c > index 20d5f80..e82e101 100644 > --- a/io/init.c > +++ b/io/init.c > @@ -23,7 +23,7 @@ > #include "init.h" > #include "io.h" > > -char *progname; > +char *io_progname; > int exitcode; > int expert; > int idlethread; > @@ -35,7 +35,7 @@ usage(void) > { > fprintf(stderr, > _("Usage: %s [-adfinrRstVx] [-m mode] [-p prog] [[-c|-C] cmd]... file\n"), > - progname); > + io_progname); > exit(1); > } > > @@ -142,7 +142,7 @@ init( > xfs_fsop_geom_t geometry = { 0 }; > struct fs_path fsp; > > - progname = basename(argv[0]); > + io_progname = basename(argv[0]); > setlocale(LC_ALL, ""); > bindtextdomain(PACKAGE, LOCALEDIR); > textdomain(PACKAGE); > @@ -186,7 +186,7 @@ init( > flags |= IO_NONBLOCK; > break; > case 'p': > - progname = optarg; > + io_progname = optarg; > break; > case 'r': > flags |= IO_READONLY; > @@ -207,7 +207,7 @@ init( > expert = 1; > break; > case 'V': > - printf(_("%s version %s\n"), progname, VERSION); > + printf(_("%s version %s\n"), io_progname, VERSION); > exit(0); > default: > usage(); > diff --git a/mdrestore/xfs_mdrestore.c b/mdrestore/xfs_mdrestore.c > index 9d1b4e8..b840a54 100644 > --- a/mdrestore/xfs_mdrestore.c > +++ b/mdrestore/xfs_mdrestore.c > @@ -19,7 +19,7 @@ > #include "libxfs.h" > #include "xfs_metadump.h" > > -char *progname; > +char *mdrestore_progname; > int show_progress = 0; > int show_info = 0; > int progress_since_warning = 0; > @@ -30,7 +30,7 @@ fatal(const char *msg, ...) > va_list args; > > va_start(args, msg); > - fprintf(stderr, "%s: ", progname); > + fprintf(stderr, "%s: ", mdrestore_progname); > vfprintf(stderr, msg, args); > exit(1); > } > @@ -194,7 +194,7 @@ perform_restore( > static void > usage(void) > { > - fprintf(stderr, "Usage: %s [-V] [-g] source target\n", progname); > + fprintf(stderr, "Usage: %s [-V] [-g] source target\n", mdrestore_progname); line > 80 cols, we try to avoid that > exit(1); > } > > @@ -212,7 +212,7 @@ main( > struct stat statbuf; > int is_target_file; > > - progname = basename(argv[0]); > + mdrestore_progname = basename(argv[0]); > > while ((c = getopt(argc, argv, "giV")) != EOF) { > switch (c) { > @@ -223,7 +223,7 @@ main( > show_info = 1; > break; > case 'V': > - printf("%s version %s\n", progname, VERSION); > + printf("%s version %s\n", mdrestore_progname, VERSION); 80 col > exit(0); > default: > usage(); > diff --git a/quota/init.c b/quota/init.c > index d45dc4c..46403de 100644 > --- a/quota/init.c > +++ b/quota/init.c > @@ -21,7 +21,7 @@ > #include "input.h" > #include "init.h" > > -char *progname; > +char *quota_progname; > int exitcode; > int expert; > bool foreign_allowed = false; > @@ -47,7 +47,7 @@ usage(void) > { > fprintf(stderr, > _("Usage: %s [-V] [-x] [-f] [-p prog] [-c cmd]... [-d project]... [path]\n"), > - progname); > + quota_progname); > exit(1); > } > > @@ -147,7 +147,7 @@ init( > { > int c; > > - progname = basename(argv[0]); > + quota_progname = basename(argv[0]); > setlocale(LC_ALL, ""); > bindtextdomain(PACKAGE, LOCALEDIR); > textdomain(PACKAGE); > @@ -173,13 +173,13 @@ init( > projid_file = optarg; > break; > case 'p': > - progname = optarg; > + quota_progname = optarg; > break; > case 'x': > expert++; > break; > case 'V': > - printf(_("%s version %s\n"), progname, VERSION); > + printf(_("%s version %s\n"), quota_progname, VERSION); > exit(0); > default: > usage(); > diff --git a/spaceman/init.c b/spaceman/init.c > index b3eface..bedf112 100644 > --- a/spaceman/init.c > +++ b/spaceman/init.c > @@ -23,7 +23,7 @@ > #include "path.h" > #include "space.h" > > -char *progname; > +char *spaceman_progname; > int exitcode; > > void > @@ -31,7 +31,7 @@ usage(void) > { > fprintf(stderr, > _("Usage: %s [-c cmd] file\n"), > - progname); > + spaceman_progname); > exit(1); > } > > @@ -74,7 +74,7 @@ init( > xfs_fsop_geom_t geometry = { 0 }; > struct fs_path fsp; > > - progname = basename(argv[0]); > + spaceman_progname = basename(argv[0]); > setlocale(LC_ALL, ""); > bindtextdomain(PACKAGE, LOCALEDIR); > textdomain(PACKAGE); > @@ -86,7 +86,7 @@ init( > add_user_command(optarg); > break; > case 'V': > - printf(_("%s version %s\n"), progname, VERSION); > + printf(_("%s version %s\n"), spaceman_progname, VERSION); 80 col > exit(0); > default: > usage(); > ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] Rename progname as it is provided by libc 2017-09-08 16:25 ` Eric Sandeen @ 2017-09-08 16:29 ` Khem Raj 2017-09-08 18:13 ` Khem Raj 1 sibling, 0 replies; 8+ messages in thread From: Khem Raj @ 2017-09-08 16:29 UTC (permalink / raw) To: Eric Sandeen; +Cc: linux-xfs On Fri, Sep 8, 2017 at 9:25 AM, Eric Sandeen <sandeen@sandeen.net> wrote: > On 9/2/17 4:54 PM, Khem Raj wrote: >> Rename local variable progname to avoid a clash with libc >> global symbols >> >> Signed-off-by: Khem Raj <raj.khem@gmail.com> >> --- >> io/init.c | 10 +++++----- >> mdrestore/xfs_mdrestore.c | 10 +++++----- >> quota/init.c | 10 +++++----- >> spaceman/init.c | 8 ++++---- >> 4 files changed, 19 insertions(+), 19 deletions(-) >> > > First of all, this doesn't build. Please don't send untested > patches... > > [LD] xfs_io > attr.o: In function `chattr_f': > /mnt/test2/git/xfsprogs-maint/io/attr.c:316: undefined reference to `progname' > /mnt/test2/git/xfsprogs-maint/io/attr.c:310: undefined reference to `progname' > /mnt/test2/git/xfsprogs-maint/io/attr.c:332: undefined reference to `progname' > /mnt/test2/git/xfsprogs-maint/io/attr.c:326: undefined reference to `progname' > attr.o: In function `chattr_callback': > /mnt/test2/git/xfsprogs-maint/io/attr.c:252: undefined reference to `progname' > attr.o:/mnt/test2/git/xfsprogs-maint/io/attr.c:255: more undefined references to `progname' follow > collect2: ld returned 1 exit status > gmake[3]: *** [xfs_io] Error 1 > gmake[2]: *** [io] Error 2 > make[1]: *** [default] Error 2 > make: *** [default] Error 2 > > > > second: is progname really a global symbol? I know of __progname, but not progname. > > i.e. this works: > > #include <stdio.h> > > extern const char *__progname; > > int > main( > int argc, > char **argv) > { > printf("progname: %s\n", __progname); > } > > # gcc -o test test.c > # ./test > progname: test > > but this doesn't: > > #include <stdio.h> > > extern const char *progname; > > int > main( > int argc, > char **argv) > { > printf("progname %s\n", progname); > } > > # gcc -o test test.c > /tmp/ccjaZrXZ.o: In function `main': > test.c:(.text+0x12): undefined reference to `progname' > collect2: ld returned 1 exit status > > > -Eric > >> diff --git a/io/init.c b/io/init.c >> index 20d5f80..e82e101 100644 >> --- a/io/init.c >> +++ b/io/init.c >> @@ -23,7 +23,7 @@ >> #include "init.h" >> #include "io.h" >> >> -char *progname; >> +char *io_progname; >> int exitcode; >> int expert; >> int idlethread; >> @@ -35,7 +35,7 @@ usage(void) >> { >> fprintf(stderr, >> _("Usage: %s [-adfinrRstVx] [-m mode] [-p prog] [[-c|-C] cmd]... file\n"), >> - progname); >> + io_progname); >> exit(1); >> } >> >> @@ -142,7 +142,7 @@ init( >> xfs_fsop_geom_t geometry = { 0 }; >> struct fs_path fsp; >> >> - progname = basename(argv[0]); >> + io_progname = basename(argv[0]); >> setlocale(LC_ALL, ""); >> bindtextdomain(PACKAGE, LOCALEDIR); >> textdomain(PACKAGE); >> @@ -186,7 +186,7 @@ init( >> flags |= IO_NONBLOCK; >> break; >> case 'p': >> - progname = optarg; >> + io_progname = optarg; >> break; >> case 'r': >> flags |= IO_READONLY; >> @@ -207,7 +207,7 @@ init( >> expert = 1; >> break; >> case 'V': >> - printf(_("%s version %s\n"), progname, VERSION); >> + printf(_("%s version %s\n"), io_progname, VERSION); >> exit(0); >> default: >> usage(); >> diff --git a/mdrestore/xfs_mdrestore.c b/mdrestore/xfs_mdrestore.c >> index 9d1b4e8..b840a54 100644 >> --- a/mdrestore/xfs_mdrestore.c >> +++ b/mdrestore/xfs_mdrestore.c >> @@ -19,7 +19,7 @@ >> #include "libxfs.h" >> #include "xfs_metadump.h" >> >> -char *progname; >> +char *mdrestore_progname; >> int show_progress = 0; >> int show_info = 0; >> int progress_since_warning = 0; >> @@ -30,7 +30,7 @@ fatal(const char *msg, ...) >> va_list args; >> >> va_start(args, msg); >> - fprintf(stderr, "%s: ", progname); >> + fprintf(stderr, "%s: ", mdrestore_progname); >> vfprintf(stderr, msg, args); >> exit(1); >> } >> @@ -194,7 +194,7 @@ perform_restore( >> static void >> usage(void) >> { >> - fprintf(stderr, "Usage: %s [-V] [-g] source target\n", progname); >> + fprintf(stderr, "Usage: %s [-V] [-g] source target\n", mdrestore_progname); > > line > 80 cols, we try to avoid that > >> exit(1); >> } >> >> @@ -212,7 +212,7 @@ main( >> struct stat statbuf; >> int is_target_file; >> >> - progname = basename(argv[0]); >> + mdrestore_progname = basename(argv[0]); >> >> while ((c = getopt(argc, argv, "giV")) != EOF) { >> switch (c) { >> @@ -223,7 +223,7 @@ main( >> show_info = 1; >> break; >> case 'V': >> - printf("%s version %s\n", progname, VERSION); >> + printf("%s version %s\n", mdrestore_progname, VERSION); > > 80 col > >> exit(0); >> default: >> usage(); >> diff --git a/quota/init.c b/quota/init.c >> index d45dc4c..46403de 100644 >> --- a/quota/init.c >> +++ b/quota/init.c >> @@ -21,7 +21,7 @@ >> #include "input.h" >> #include "init.h" >> >> -char *progname; >> +char *quota_progname; >> int exitcode; >> int expert; >> bool foreign_allowed = false; >> @@ -47,7 +47,7 @@ usage(void) >> { >> fprintf(stderr, >> _("Usage: %s [-V] [-x] [-f] [-p prog] [-c cmd]... [-d project]... [path]\n"), >> - progname); >> + quota_progname); >> exit(1); >> } >> >> @@ -147,7 +147,7 @@ init( >> { >> int c; >> >> - progname = basename(argv[0]); >> + quota_progname = basename(argv[0]); >> setlocale(LC_ALL, ""); >> bindtextdomain(PACKAGE, LOCALEDIR); >> textdomain(PACKAGE); >> @@ -173,13 +173,13 @@ init( >> projid_file = optarg; >> break; >> case 'p': >> - progname = optarg; >> + quota_progname = optarg; >> break; >> case 'x': >> expert++; >> break; >> case 'V': >> - printf(_("%s version %s\n"), progname, VERSION); >> + printf(_("%s version %s\n"), quota_progname, VERSION); >> exit(0); >> default: >> usage(); >> diff --git a/spaceman/init.c b/spaceman/init.c >> index b3eface..bedf112 100644 >> --- a/spaceman/init.c >> +++ b/spaceman/init.c >> @@ -23,7 +23,7 @@ >> #include "path.h" >> #include "space.h" >> >> -char *progname; >> +char *spaceman_progname; >> int exitcode; >> >> void >> @@ -31,7 +31,7 @@ usage(void) >> { >> fprintf(stderr, >> _("Usage: %s [-c cmd] file\n"), >> - progname); >> + spaceman_progname); >> exit(1); >> } >> >> @@ -74,7 +74,7 @@ init( >> xfs_fsop_geom_t geometry = { 0 }; >> struct fs_path fsp; >> >> - progname = basename(argv[0]); >> + spaceman_progname = basename(argv[0]); >> setlocale(LC_ALL, ""); >> bindtextdomain(PACKAGE, LOCALEDIR); >> textdomain(PACKAGE); >> @@ -86,7 +86,7 @@ init( >> add_user_command(optarg); >> break; >> case 'V': >> - printf(_("%s version %s\n"), progname, VERSION); >> + printf(_("%s version %s\n"), spaceman_progname, VERSION); > > 80 col > >> exit(0); >> default: >> usage(); >> ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] Rename progname as it is provided by libc 2017-09-08 16:25 ` Eric Sandeen 2017-09-08 16:29 ` Khem Raj @ 2017-09-08 18:13 ` Khem Raj 1 sibling, 0 replies; 8+ messages in thread From: Khem Raj @ 2017-09-08 18:13 UTC (permalink / raw) To: Eric Sandeen; +Cc: linux-xfs Hi Eric On Fri, Sep 8, 2017 at 9:25 AM, Eric Sandeen <sandeen@sandeen.net> wrote: > On 9/2/17 4:54 PM, Khem Raj wrote: >> Rename local variable progname to avoid a clash with libc >> global symbols >> >> Signed-off-by: Khem Raj <raj.khem@gmail.com> >> --- >> io/init.c | 10 +++++----- >> mdrestore/xfs_mdrestore.c | 10 +++++----- >> quota/init.c | 10 +++++----- >> spaceman/init.c | 8 ++++---- >> 4 files changed, 19 insertions(+), 19 deletions(-) >> > > First of all, this doesn't build. Please don't send untested > patches... I am cross building with clang and for musl systems and the cross build succeeds for me, I realized, I should have done native builds too sorry about that as discussed in previous replies, the symbols are conflicting from local libraries in xfsprogs unlike it coming from libc as I was thinking earlier. I am building with --disable-static thats the root cause of issue I am seeing because its finding two progname symbols during link one from libxfs.so and another one from init.o when linking the binary e.g. xfs_io I wonder if building with --disable-static is valid/supported configuration for the project or not. here is error I was seeing /mnt/a/oe/build/tmp/work/cortexa7t2hf-neon-vfpv4-bec-linux-musleabi/xfsprogs/4.12.0-r0/recipe-sysroot-native/usr/bin/arm-bec-linux-musleabi/arm-bec-linux-musl eabi-ld: xfs_mdrestore.o: relocation R_ARM_REL32 against external or undefined symbol `progname' can not be used when making a PIE executable; recompile with -f PIC | /mnt/a/oe/build/tmp/work/cortexa7t2hf-neon-vfpv4-bec-linux-musleabi/xfsprogs/4.12.0-r0/recipe-sysroot-native/usr/bin/arm-bec-linux-musleabi/arm-bec-linux-musl eabi-ld: xfs_mdrestore.o(.text+0x64c): unresolvable R_ARM_REL32 relocation against symbol `progname' shared libs e.g. libxfs.so are not built with -fPIC if I added -fPIC to CFLAGS and now compiler is happy. > > [LD] xfs_io > attr.o: In function `chattr_f': > /mnt/test2/git/xfsprogs-maint/io/attr.c:316: undefined reference to `progname' > /mnt/test2/git/xfsprogs-maint/io/attr.c:310: undefined reference to `progname' > /mnt/test2/git/xfsprogs-maint/io/attr.c:332: undefined reference to `progname' > /mnt/test2/git/xfsprogs-maint/io/attr.c:326: undefined reference to `progname' > attr.o: In function `chattr_callback': > /mnt/test2/git/xfsprogs-maint/io/attr.c:252: undefined reference to `progname' > attr.o:/mnt/test2/git/xfsprogs-maint/io/attr.c:255: more undefined references to `progname' follow > collect2: ld returned 1 exit status > gmake[3]: *** [xfs_io] Error 1 > gmake[2]: *** [io] Error 2 > make[1]: *** [default] Error 2 > make: *** [default] Error 2 > > > > second: is progname really a global symbol? I know of __progname, but not progname. > > i.e. this works: > > #include <stdio.h> > > extern const char *__progname; > > int > main( > int argc, > char **argv) > { > printf("progname: %s\n", __progname); > } > > # gcc -o test test.c > # ./test > progname: test > > but this doesn't: > > #include <stdio.h> > > extern const char *progname; > > int > main( > int argc, > char **argv) > { > printf("progname %s\n", progname); > } > > # gcc -o test test.c > /tmp/ccjaZrXZ.o: In function `main': > test.c:(.text+0x12): undefined reference to `progname' > collect2: ld returned 1 exit status > > > -Eric > >> diff --git a/io/init.c b/io/init.c >> index 20d5f80..e82e101 100644 >> --- a/io/init.c >> +++ b/io/init.c >> @@ -23,7 +23,7 @@ >> #include "init.h" >> #include "io.h" >> >> -char *progname; >> +char *io_progname; >> int exitcode; >> int expert; >> int idlethread; >> @@ -35,7 +35,7 @@ usage(void) >> { >> fprintf(stderr, >> _("Usage: %s [-adfinrRstVx] [-m mode] [-p prog] [[-c|-C] cmd]... file\n"), >> - progname); >> + io_progname); >> exit(1); >> } >> >> @@ -142,7 +142,7 @@ init( >> xfs_fsop_geom_t geometry = { 0 }; >> struct fs_path fsp; >> >> - progname = basename(argv[0]); >> + io_progname = basename(argv[0]); >> setlocale(LC_ALL, ""); >> bindtextdomain(PACKAGE, LOCALEDIR); >> textdomain(PACKAGE); >> @@ -186,7 +186,7 @@ init( >> flags |= IO_NONBLOCK; >> break; >> case 'p': >> - progname = optarg; >> + io_progname = optarg; >> break; >> case 'r': >> flags |= IO_READONLY; >> @@ -207,7 +207,7 @@ init( >> expert = 1; >> break; >> case 'V': >> - printf(_("%s version %s\n"), progname, VERSION); >> + printf(_("%s version %s\n"), io_progname, VERSION); >> exit(0); >> default: >> usage(); >> diff --git a/mdrestore/xfs_mdrestore.c b/mdrestore/xfs_mdrestore.c >> index 9d1b4e8..b840a54 100644 >> --- a/mdrestore/xfs_mdrestore.c >> +++ b/mdrestore/xfs_mdrestore.c >> @@ -19,7 +19,7 @@ >> #include "libxfs.h" >> #include "xfs_metadump.h" >> >> -char *progname; >> +char *mdrestore_progname; >> int show_progress = 0; >> int show_info = 0; >> int progress_since_warning = 0; >> @@ -30,7 +30,7 @@ fatal(const char *msg, ...) >> va_list args; >> >> va_start(args, msg); >> - fprintf(stderr, "%s: ", progname); >> + fprintf(stderr, "%s: ", mdrestore_progname); >> vfprintf(stderr, msg, args); >> exit(1); >> } >> @@ -194,7 +194,7 @@ perform_restore( >> static void >> usage(void) >> { >> - fprintf(stderr, "Usage: %s [-V] [-g] source target\n", progname); >> + fprintf(stderr, "Usage: %s [-V] [-g] source target\n", mdrestore_progname); > > line > 80 cols, we try to avoid that > >> exit(1); >> } >> >> @@ -212,7 +212,7 @@ main( >> struct stat statbuf; >> int is_target_file; >> >> - progname = basename(argv[0]); >> + mdrestore_progname = basename(argv[0]); >> >> while ((c = getopt(argc, argv, "giV")) != EOF) { >> switch (c) { >> @@ -223,7 +223,7 @@ main( >> show_info = 1; >> break; >> case 'V': >> - printf("%s version %s\n", progname, VERSION); >> + printf("%s version %s\n", mdrestore_progname, VERSION); > > 80 col > >> exit(0); >> default: >> usage(); >> diff --git a/quota/init.c b/quota/init.c >> index d45dc4c..46403de 100644 >> --- a/quota/init.c >> +++ b/quota/init.c >> @@ -21,7 +21,7 @@ >> #include "input.h" >> #include "init.h" >> >> -char *progname; >> +char *quota_progname; >> int exitcode; >> int expert; >> bool foreign_allowed = false; >> @@ -47,7 +47,7 @@ usage(void) >> { >> fprintf(stderr, >> _("Usage: %s [-V] [-x] [-f] [-p prog] [-c cmd]... [-d project]... [path]\n"), >> - progname); >> + quota_progname); >> exit(1); >> } >> >> @@ -147,7 +147,7 @@ init( >> { >> int c; >> >> - progname = basename(argv[0]); >> + quota_progname = basename(argv[0]); >> setlocale(LC_ALL, ""); >> bindtextdomain(PACKAGE, LOCALEDIR); >> textdomain(PACKAGE); >> @@ -173,13 +173,13 @@ init( >> projid_file = optarg; >> break; >> case 'p': >> - progname = optarg; >> + quota_progname = optarg; >> break; >> case 'x': >> expert++; >> break; >> case 'V': >> - printf(_("%s version %s\n"), progname, VERSION); >> + printf(_("%s version %s\n"), quota_progname, VERSION); >> exit(0); >> default: >> usage(); >> diff --git a/spaceman/init.c b/spaceman/init.c >> index b3eface..bedf112 100644 >> --- a/spaceman/init.c >> +++ b/spaceman/init.c >> @@ -23,7 +23,7 @@ >> #include "path.h" >> #include "space.h" >> >> -char *progname; >> +char *spaceman_progname; >> int exitcode; >> >> void >> @@ -31,7 +31,7 @@ usage(void) >> { >> fprintf(stderr, >> _("Usage: %s [-c cmd] file\n"), >> - progname); >> + spaceman_progname); >> exit(1); >> } >> >> @@ -74,7 +74,7 @@ init( >> xfs_fsop_geom_t geometry = { 0 }; >> struct fs_path fsp; >> >> - progname = basename(argv[0]); >> + spaceman_progname = basename(argv[0]); >> setlocale(LC_ALL, ""); >> bindtextdomain(PACKAGE, LOCALEDIR); >> textdomain(PACKAGE); >> @@ -86,7 +86,7 @@ init( >> add_user_command(optarg); >> break; >> case 'V': >> - printf(_("%s version %s\n"), progname, VERSION); >> + printf(_("%s version %s\n"), spaceman_progname, VERSION); > > 80 col > >> exit(0); >> default: >> usage(); >> ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2017-09-08 18:14 UTC | newest] Thread overview: 8+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2017-09-02 21:54 [PATCH] Rename progname as it is provided by libc Khem Raj 2017-09-02 22:20 ` Dave Chinner 2017-09-02 23:42 ` Khem Raj 2017-09-03 7:21 ` Khem Raj 2017-09-03 7:48 ` Christoph Hellwig 2017-09-08 16:25 ` Eric Sandeen 2017-09-08 16:29 ` Khem Raj 2017-09-08 18:13 ` Khem Raj
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox