Comment 9 for bug 1842924

Revision history for this message
Joseph Gooch (whitemystic) wrote :

I removed
#if HAVE_GETFSSTAT
And
#endif /* HAVE_GETFSSTAT */

Based on looking at the Check_HR_FileSys_NFS function earlier in the file, it was structured like:
#if HAVE_GETFSSTAT
  something that looks nothing like the autofs function above with different constants
#else
  something that looks just like the autofs function above
#endif

BUT, I'm not sure if there should be a HAVE_GETFSSTAT section in this function, and if so what it should be. I'm basically poking around and compiling and doing a lot of trial and error. I played enough in GDB to determine that netsnmp_fsys_arch_load IS finding all the correct mountpoints, but I couldn't directly determine at what point the physical ones were being thrown away. The constants all over the code made it very difficult to debug with GDB as it couldn't resolve the precompiler directives, and I very quickly got sucked into a rabbit hole of header spelunking.

I pinned the previous release on my servers to fix the immediate issue. I may have more time to look later, but it's very likely it'll take the original patch author, or someone who has any experience working in the daemon far less time to resolve :)