Hi Seiji, I see your point, but when I introduced this fread() return code checks I intended to make sure fread() had read the amount of data I said it to, so the fix would be exchange the nmemb and size fields. Let me know what you think.
Thanks, Rajiv Andrade IBM LTC Security Development
0001-Fixed-fread-return-value-check.patch
Description: Binary data
On Feb 13, 2010, at 9:50 PM, Seiji Munetoh wrote: > Rajiv, > > Fix for tcs_evlog_imaem.c. > The return of fread must be 1 since it use nmemb =1. > and changed "int" to UINT32 to make sure some length (4 bytes). > I tested this with Fedora 12, 2.6.31.12-174.2.3.fc12.x86_64 kernel. > > Signed-off-by: Seiji Munetoh <[email protected]> > > regards, > -- > Seiji Munetoh > <IMA_eventlog.patch>------------------------------------------------------------------------------ > SOLARIS 10 is the OS for Data Centers - provides features such as DTrace, > Predictive Self Healing and Award Winning ZFS. Get Solaris 10 NOW > http://p.sf.net/sfu/solaris-dev2dev_______________________________________________ > TrouSerS-tech mailing list > [email protected] > https://lists.sourceforge.net/lists/listinfo/trousers-tech
------------------------------------------------------------------------------ Download Intel® Parallel Studio Eval Try the new software tools for yourself. Speed compiling, find bugs proactively, and fine-tune applications for parallel performance. See why Intel Parallel Studio got high marks during beta. http://p.sf.net/sfu/intel-sw-dev
_______________________________________________ TrouSerS-tech mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/trousers-tech
