[PATCH v2] zic, various tests: use 64-bit stat/ftw/readdir functions
Nick Alcock
nix@esperi.org.uk
Mon Jun 26 17:47:00 GMT 2017
On 26 Jun 2017, Joseph Myers said:
> On Mon, 26 Jun 2017, Nick Alcock wrote:
>
>> Index: glibc/io/bug-ftw2.c
>> ===================================================================
>> --- glibc.orig/io/bug-ftw2.c 2017-06-26 11:32:54.460712810 +0100
>> +++ glibc/io/bug-ftw2.c 2017-06-26 11:33:38.730559302 +0100
>> @@ -69,7 +69,7 @@
>> {
>> mtrace ();
>>
>> - ftw (".", callback, 10);
>> + ftw64 (".", callback, 10);
>>
>> if (! sawcur)
>> {
>
> A test of ftw should arguably be using ftw exactly, not ftw64, and
> probably be paired with a corresponding tests (using a shared test
> skeleton) that tests the same things for ftw64. I.e., adding appropriate
> checks so this test doesn't fail for 64-bit inode numbers might be better,
> to avoid changing what function this test tests.
i.e. UNSUPPORTED-out this test iff it EOVERFLOWs, and have another
bug-ftw64-2.c that uses ftw64 instead and is run unconditionally?
(And similarly for the other non-64 ftw tests, probably all using the
usual macro-expansion-and-#include trick to keep code duplication down.)
Makes considerable sense: I'll look at that shortly.
More information about the Libc-alpha
mailing list