[Bro] Is it is a code issue about bro-2.3?

[Robert_Yang at trendmicro.com.cn]

Dear,

BroFile::Write() in src/File.cc,

len = fwrite(data, 1, len, f);
if ( len <=0 )
return false;

Maybe, the method to check the return value of fwrite is incorrect. We should call ferror to check file operation's result. The following is fixing.

        len = fwrite(data, 1, len, f);
        if ( ferror(f) ) {
             clearerr(f);
             return false;
        }

Would you please verify this question?

Thanks!


<table class="TM_EMAIL_NOTICE"><tr><td><pre>
TREND MICRO EMAIL NOTICE
The information contained in this email and any attachments is confidential 
and may be subject to copyright or other intellectual property protection. 
If you are not the intended recipient, you are not authorized to use or 
disclose this information, and we request that you notify us by reply mail or
telephone and delete the original message from your mail system.
</pre></td></tr></table>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mailman.ICSI.Berkeley.EDU/pipermail/bro/attachments/20141028/3300cbc7/attachment.html