<html>
<head>
<style><!--
.hmmessage P
{
margin:0px;
padding:0px
}
body.hmmessage
{
font-size: 10pt;
font-family:Tahoma
}
--></style></head>
<body class='hmmessage'><div dir='ltr'>
<br><div>hi, new patch :).<br><br>Apparently was required also monitor_config_file_fallback().<br><br>> Hi Ariel,<br>> <br>> actually I reread the patch and I think there is a simpler way to fix<br>> the ticket..directly in the monitor_config_file() function, there is<br>> already a stat() call:<br>> 1823 ret = stat(file, &file_stat);<br>> 1824 if (ret < 0) {<br>> 1825 err = errno;<br>> 1826 DEBUG(0, ("Could not stat file [%s]. Error [%d:%s]\n",<br>> 1827 file, err, strerror(err)));<br>> 1828 return err;<br>> 1829 }<br>> <br>> <br>> So I think that simply extending the error handler with special-casing<br>> ENOENT as you had in your patch would work fine:<br>> + if (ret == ENOENT) {<br>> + DEBUG(SSSDBG_MINOR_FAILURE,<br>> + ("file [%s] is missing, but will skip this check.\n",<br>> + RESOLV_CONF_PATH));<br>> + } else {<br>> + DEBUG(SSSDBG_MINOR_FAILURE,("Could not stat file [%s]. Error [%d:%s]\n",<br>> + RESOLV_CONF_PATH, ret, strerror(ret)));<br>> + }<br>> <br>> _______________________________________________<br>> sssd-devel mailing list<br>> sssd-devel@lists.fedorahosted.org<br>> https://lists.fedorahosted.org/mailman/listinfo/sssd-devel<br></div>                                            </div></body>
</html>