Hi,

yes, you're right it should be /proc/mounts sorry for mistake.
I'll try to look at the source code once again and try to prepare recommended 
modification.

Josef

----- Original Message -----
| From: "Bart Van Assche" <bvanass...@acm.org>
| To: "Josef Ridky" <jri...@redhat.com>, "net-snmp-coders" 
<net-snmp-coders@lists.sourceforge.net>
| Sent: Wednesday, June 20, 2018 4:18:56 PM
| Subject: Re: [PATCH / request for votes] Cannot statfs net:[********]#***: No 
such file or directory logged twice per
| every 10 minutes
| 
| On 06/20/18 02:46, Josef Ridky wrote:
| > Error log is written twice when processing /proc/mtab and calling
| > statfs() for each line in it.
| 
| Did you mean /etc/mtab or /proc/mounts? I'm not aware of any /proc/mtab
| file on Linux.
| 
| > It stumbles upon this line:
| > 
| > proc net:[4026532288] proc rw,nosuid,nodev,noexec,relatime 0 0
| > 
| > Looking into the code, "Cannot statfs %s" is logged with "error" level.
| > snmpd should log the error only once.
| 
| Although the patch itself looks fine to me: should the mount table
| parser be updated such that it understands that line?
| 
| Thanks,
| 
| Bart.
| 
| 
| 
| 

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Net-snmp-coders mailing list
Net-snmp-coders@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/net-snmp-coders

Reply via email to