diff options
author | Corinna Vinschen <corinna@vinschen.de> | 2024-04-09 16:52:58 +0200 |
---|---|---|
committer | Corinna Vinschen <corinna@vinschen.de> | 2024-04-09 17:02:35 +0200 |
commit | 2855c35c41ed5c950e40ce0ae0d99d00185e05ad (patch) | |
tree | 69e4670767938614d8a6721131e6b11529c22010 /winsup/cygwin/fhandler/procnet.cc | |
parent | 38b51398359076eed01d8f1b0f9c061d16a658f5 (diff) | |
download | newlib-2855c35c41ed5c950e40ce0ae0d99d00185e05ad.zip newlib-2855c35c41ed5c950e40ce0ae0d99d00185e05ad.tar.gz newlib-2855c35c41ed5c950e40ce0ae0d99d00185e05ad.tar.bz2 |
Cygwin: fhandler_virtual::exists: always set fileid
Commit a0a25849f9dd ("Cygwin: fhandler_virtual: move fileid to path_conv
member") broke `ls -l /proc/<PID>'. Turns out, the commit forgot to set
the fileid in case of a virtual root dir (i. e., returning virt_rootdir)
in fhandlers utilizing fileid. This crashed opendir() due to a random
fileid.
Make sure to set fileid in any case.
Fixes: a0a25849f9dd ("Cygwin: fhandler_virtual: move fileid to path_conv member")
Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
Diffstat (limited to 'winsup/cygwin/fhandler/procnet.cc')
-rw-r--r-- | winsup/cygwin/fhandler/procnet.cc | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/winsup/cygwin/fhandler/procnet.cc b/winsup/cygwin/fhandler/procnet.cc index 112aee8..631fe8c 100644 --- a/winsup/cygwin/fhandler/procnet.cc +++ b/winsup/cygwin/fhandler/procnet.cc @@ -48,7 +48,10 @@ fhandler_procnet::exists () while (*path != 0 && !isdirsep (*path)) path++; if (*path == 0) - return virt_rootdir; + { + fileid () = 0; + return virt_rootdir; + } virt_tab_t *entry = virt_tab_search (path + 1, false, procnet_tab, PROCNET_LINK_COUNT); @@ -59,6 +62,7 @@ fhandler_procnet::exists () fileid () = entry - procnet_tab; return entry->type; } + fileid () = -1; return virt_none; } |