summaryrefslogtreecommitdiffstats
path: root/lib/Plugins/KerneloopsScanner.cpp
Commit message (Collapse)AuthorAgeFilesLines
* moved LoadSettings from plugins into PluginManagerZdenek Prikryl2009-08-111-8/+0
|
* add the GPL notice to the kerneloops' scannerAnton Arapov2009-08-111-0/+26
| | | | this is important, since the codebase of the kerneloops.org was used.
* kerneloops eliminate commented code, related to type distinguishing...Anton Arapov2009-08-111-2/+2
| | | | it seems stable, proved by the stress tests. :)
* Kerneloops plugin: style fixesAnton Arapov2009-08-101-117/+123
|
* Split real code from lib/MiddleWare/Plugin.h into Plugin.cpp.Denys Vlasenko2009-08-071-3/+0
| | | | | | 30k smaller code. Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
* CDebugDump::Create(pDir, pUID): make 2nd param uid_t, not stringDenys Vlasenko2009-08-051-1/+1
|
* dumpoops: hopefully fix KerneloopsScanner plugin usageDenys Vlasenko2009-08-041-0/+15
| | | | Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
* delete semicolon in "if (expr); stmt;" , autodetect dmesg/syslogDenys Vlasenko2009-07-311-2/+2
| | | | Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
* added new interface for geting plugins' settings (will be used in gui)Zdenek Prikryl2009-07-311-4/+17
|
* Move PLUGIN_INFOs to .cpp files: same object must never be defined twiceDenys Vlasenko2009-07-311-0/+9
| | | | | | | | 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>
* KerneloopsReporter: plug a leak in writefunctionDenys Vlasenko2009-07-311-11/+18
| | | | | | Also some preparatory cleanups for future dumpoops patch are included Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
* rename auto variables to not have m_ prefixes - they are not members!Denys Vlasenko2009-07-311-19/+16
| | | | | | no logic changes Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
* bug fixes in kerneloops scannerDenys Vlasenko2009-07-311-54/+45
| | | | | | | | | | | | | | | | | | | | | 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>
* fixed security issueZdenek Prikryl2009-06-301-2/+1
| | | | User can read only his debugdump directories
* Handle logs with NUL chars betterMichal Schmidt2009-06-201-4/+4
| | | | | | | | | | | It is not too rare that '\0' chars appear in /var/log/messages. I saw a real-life case where kerneloops would show a popup with the same old oops after every login. It turned out that there were NUL chars in the log which prevented kerneloops from seeing its marker, so it always treated the old oops in the log as new. This patch fixes it by always going through the whole known length of the buffer (not stopping on NUL chars) and using less string-oriented functions in fill_lineinfo().
* Really limit max size of log to read to 32 MBMichal Schmidt2009-06-201-3/+3
| | | | | Because of MAX/MIN confusion, the buffer size was always at least 32 MB, instead of at most 32 MB as intended.
* new scanner, it is used by abrt's cronZdenek Prikryl2009-05-121-0/+159