@bbockelm commented on this pull request.
In src/XrdVoms/XrdVomsMapfile.cc:
> + if (rval > 0) { + // Woke up early due to a signal; re-run prior logic. + continue; + } + next_update = monotonic_time_s() + m_update_interval; + struct stat statbuf; + if (-1 == stat(myself->m_mapfile.c_str(), &statbuf)) { + myself->m_edest->Emsg("XrdVomsMapfile", errno, "Error checking the mapfile", + myself->m_mapfile.c_str()); + myself->m_mapfile_ctime.tv_sec = 0; + myself->m_mapfile_ctime.tv_nsec = 0; + myself->m_is_valid = false; + continue; + } + if ((myself->m_mapfile_ctime.tv_sec == statbuf.st_ctim.tv_sec) && + (myself->m_mapfile_ctime.tv_nsec == statbuf.st_ctim.tv_nsec))
Yup - ctime
can't be manipulated by userspace tools, making it more reliable for detecting changes rolled out by some external tooling or renames. Added the comment on this!
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you are subscribed to this thread.
Use REPLY-ALL to reply to list
To unsubscribe from the XROOTD-DEV list, click the following link:
https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=XROOTD-DEV&A=1