bug fixes in kerneloops scanner

CKerneloopsScanner::Run - scan syslog file first, then dmesg
(was other way around)

CKerneloopsScanner::SaveOopsToDebug - cast time_t and size_t to long
when we feed them to snprintf

CKerneloopsScanner::ScanSysLogFile - do not seek to negative offsets,
 POSIX does not allow that

CSysLog::QueueOops - plug memory leak

Also used some abrtlib functions where appropriate, use open instead of fopen
where appropriate, do not check error in time() - it never fails,
move around bits of code in CKerneloopsScanner
to make it easier in future to create a tool for parsing
arbitrary files for oopses.

Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>

1 files changed, 2 insertions, 3 deletions

diff --git a/lib/Plugins/KerneloopsScanner.h b/lib/Plugins/KerneloopsScanner.h
index 2f28239..08f8c6c 100644
--- a/lib/Plugins/KerneloopsScanner.h
+++ b/lib/Plugins/KerneloopsScanner.h
@@ -13,9 +13,8 @@ class CKerneloopsScanner : public CAction
         bool m_bSysLogFileScanned;
 
         void SaveOopsToDebugDump();
-        void ScanDmesg();
-        void ScanSysLogFile(const char *filename, int issyslog);
-        void WriteSysLog(int m_nCount);
+        int ScanDmesg();
+        int ScanSysLogFile(const char *filename, int issyslog);
     public:
         CKerneloopsScanner() :
             m_sSysLogFile("/var/log/messages"),