diff options
| author | Guy Harris <gharris@sonic.net> | 2022-01-19 23:09:31 -0800 |
|---|---|---|
| committer | Guy Harris <gharris@sonic.net> | 2022-01-19 23:09:31 -0800 |
| commit | f570e6be7c5573f6e9310db751ede041a5e6151f (patch) | |
| tree | 209ab231cfff45cf23078c61766ba387fe33570e /testprogs | |
| parent | bfb4425464228515d49ffd1f643908598e5c58ab (diff) | |
Revert "findalldevstest: temporary test to see if this squelches Valgrind warnings."
This reverts commit 778f2c0e806090dc524627161bca826cc1d061a2.
Yes, Illumos (and probably Solaris) may fflush() all the standard I/O
streams before exiting, but it doesn't fclose() them, so stuff might not
be freed before exiting (but it should be accessible from stdin, stdout,
or stderr, so it's at worst a "still reachable" issue - what gives?).
There *appears* to be a suppressor for that in the Solaris valgrind, but
it doesn't seem to be working.
Diffstat (limited to 'testprogs')
| -rw-r--r-- | testprogs/findalldevstest.c | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/testprogs/findalldevstest.c b/testprogs/findalldevstest.c index 4e2df212..06293209 100644 --- a/testprogs/findalldevstest.c +++ b/testprogs/findalldevstest.c @@ -183,11 +183,6 @@ int main(int argc _U_, char **argv _U_) } pcap_freealldevs(alldevs); -// -// XXX - temporary hack to see if this squelches Valgrind warnings -// on Illumos about leaks from stdio. -// -fclose(stdout); exit(exit_status); } |