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

Robert_Yang at trendmicro.com.cn Robert_Yang at trendmicro.com.cn
Mon Oct 27 23:55:53 PDT 2014


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) ) {
             return false;

Would you please verify this question?


<table class="TM_EMAIL_NOTICE"><tr><td><pre>
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.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mailman.ICSI.Berkeley.EDU/pipermail/bro/attachments/20141028/3300cbc7/attachment.html 

More information about the Bro mailing list