this is important, since the codebase of the kerneloops.org was used.

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

and if structure is defined in a .h file, that happens.
Since this particular structure has non-trivial destructor,
it was running twice and resulted in double-free.

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

Also some preparatory cleanups for future dumpoops patch are included

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

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>