Unneeded compiler warning in glob.c

[Tim Rühsen]

Here is a patch to silence this warning:

glob.c: In function 'rpl_glob':
glob.c:618:64: warning: pointer of type 'void *' used in arithmetic
[-Wpointer-arith]
err = getpwnam_r (s.data, &pwbuf, s.data + ssize,


With Best Regards, Tim

[Paul Eggert]

Post by Tim Rühsen
glob.c:618:64: warning: pointer of type 'void *' used in arithmetic
[-Wpointer-arith]
err = getpwnam_r (s.data, &pwbuf, s.data + ssize,
With Best Regards, Tim

Thanks, I tweaked it slightly to avoid inserting a cast (as it's better
to avoid casts when it's easy), and installed the attached patch into
Gnulib, in your name. I'll CC: Adhemerval as this should go into glibc
too, at some point.

[Bruno Haible]

Post by Paul Eggert

Post by Tim Rühsen
glob.c:618:64: warning: pointer of type 'void *' used in arithmetic
[-Wpointer-arith]
err = getpwnam_r (s.data, &pwbuf, s.data + ssize,
With Best Regards, Tim

Thanks, I tweaked it slightly to avoid inserting a cast (as it's better
to avoid casts when it's easy), and installed the attached patch into
Gnulib, in your name.

... and I've added the ChangeLog entry for it.

But, when we look at the definition

struct scratch_buffer {
void *data; /* Pointer to the beginning of the scratch area. */
size_t length; /* Allocated space at the data pointer, in bytes. */
...
};

it seems quite natural to do pointer arithmetic on 'data'. Namely, this will
happen each time a scratch_buffer gets booked by several (not just one)
pieces of memory.

Therefore I would suggest to change the type of the 'data' field to 'char *'
in gnulib. I know that in glibc this is not needed, because glibc assumes GCC
and GCC supports 'void *' arithmetic as an extension.

Rationale: 'scratch_buffer' is a module of its own right in gnulib, and
may get more uses, in order to avoid stack overflows.

Bruno

[Paul Eggert]

Post by Bruno Haible
Therefore I would suggest to change the type of the 'data' field to 'char *'
in gnulib. I know that in glibc this is not needed, because glibc assumes GCC
and GCC supports 'void *' arithmetic as an extension.
Rationale: 'scratch_buffer' is a module of its own right in gnulib, and
may get more uses, in order to avoid stack overflows.

I think I'd rather leave it alone. glibc is moving to a more-general
module that addresses this problem in a better way, and I'd rather we
spend our limited resources doing that than forking.

[Bruno Haible]

Post by Bruno Haible
struct scratch_buffer {
void *data; /* Pointer to the beginning of the scratch area. */
size_t length; /* Allocated space at the data pointer, in bytes. */
...
};
it seems quite natural to do pointer arithmetic on 'data'. Namely, this will
happen each time a scratch_buffer gets booked by several (not just one)
pieces of memory.

Oops. scratch_buffer is not appropriate for this kind of use, as it does
not contain a 'size_t used;' field (like linebuffer does).

I was thinking at a combination between scratch_buffer and the 'alloca_used'
logic from glob.c. But this is beyond what scratch_buffer does.

Bruno

[Adhemerval Zanella]

Post by Tim Rühsen
glob.c:618:64: warning: pointer of type 'void *' used in arithmetic
[-Wpointer-arith]
                        err = getpwnam_r (s.data, &pwbuf, s.data + ssize,
With Best Regards, Tim

Thanks, I tweaked it slightly to avoid inserting a cast (as it's better to avoid casts when it's easy), and installed the attached patch into Gnulib, in your name. I'll CC: Adhemerval as this should go into glibc too, at some point.

Thanks for the heads up, I will sync it on glibc side.