summaryrefslogtreecommitdiffstats
path: root/lib/plugins
Commit message (Collapse)AuthorAgeFilesLines
* move files from lib/plugins to src/pluginsJiri Moskovcak2010-11-1053-5772/+0
|
* Decouple settings handling from old-style pluginsDenys Vlasenko2010-11-108-5/+19
| | | | | | | | The breakage was discovered when i removed Logger class. it turned out the fix is somewhat involved. This change implements it as discussed with the rest of the team. Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* remove CLogger class, abrt-event.conf + abrt-action-print superseded itDenys Vlasenko2010-11-084-188/+1
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* remove RunApp plugin, abrt-event.conf superseded itDenys Vlasenko2010-11-085-162/+0
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* abrtd: convert reporting step to run_action, part 1 (main)Denys Vlasenko2010-11-036-284/+0
| | | | | | | | | | There are a few things which are missing in this commit: for example, the list of possible reporting "paths" is not extracted from abrt_event.conf yet. Part 2 will deal with it. Tested: reporting thru bugzilla and thru logger. Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* switch "analyze" step processing to abrt_event.conf-driven wayDenys Vlasenko2010-11-034-297/+0
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* remove AutoReportUIDs supportDenys Vlasenko2010-11-021-6/+0
| | | | | | | | abrt_action.conf provides a way to configure that behaviour. This change adds a commented-out example how to do it for kerneloops. Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* introduce abrt_action.conf; use it for post-create stepDenys Vlasenko2010-11-016-58/+0
| | | | | | | | | | | | | | | | | | | | | | | | This patch replaces only the post-create step with new logic (the step which happens when abrtd detects fresh crash dump dir and needs to decide what to do with it), but it turns out this step is one of hard ones: it needs special handling of UUID. So a good chunk of hard-ish stuff is already in this patch. It also contains logic to collect the log from actions, even though so far it is simply logged. Other steps (like reporting step) will pipe it to clients. post-create step simply has no client to pipe output to. But the code is there already. GetLocalUUID() members in all plugins are unused now and are deleted from all classes. Next patches will move run_event() function into libABRT, so that it can be used from e.g. abrt-handle-crashdump utility if/when we will write it, from clients directly and so on. For now, it lives in MiddleWare.cpp since it is only used there. Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* abrtd: prepearatory patch: is_debug_dump_saved shouldn't need uidDenys Vlasenko2010-11-012-0/+22
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* new action tool: abrt-action-kerneloopsDenys Vlasenko2010-10-254-104/+95
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* introduce and use xmalloc_fgets/fgetlineDenys Vlasenko2010-10-226-34/+29
| | | | | | | This fixes problems of having long lines truncated - and we do have very long lines sometimes - curl errors with HTML, list of debuginfos etc. Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* Kerneloops should use abrt-action-analyze-oops, not -python!Denys Vlasenko2010-10-211-2/+2
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* move logger reporting to a separate program (abrt-action-print)Denys Vlasenko2010-10-202-29/+73
| | | | | | | | | | | | | | | | | Usage: abrt-action-print [v] -d DIR Print information about the crash to standard output -v, --verbose be verbose -d DIR Crash dump directory As you can see, it simply writes crash dump information to stdout, not to a file. Logger plugin spawns a child abrt-action-print -d DIR and pipes its output to log file. Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* move rhtsupport reporting to a separate program (abrt-action-rhtsupport)Denys Vlasenko2010-10-203-277/+85
| | | | | | | | | | | | | | | | The tool works similarly to abrt-action-bugzilla: Usage: abrt-action-rhtsupport -c CONFFILE -d DIR [-vs] Report a crash to RHTSupport Options: -c FILE Configuration file (may be given many times) -d DIR Crash dump directory -v Verbose -s Log to syslog Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* move uuid/duphash generation for Kerneloops to abrt-action-analyze-oopsDenys Vlasenko2010-10-191-104/+41
| | | | | | | | | | | | Usage: abrt-action-analyze-oops [-vs] -d DIR Calculates and saves UUID and DUPHASH of oops crash dumps -v, --verbose be verbose -d DIR Crash dump directory -s Log to syslog Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* move uuid/duphash generation for python to abrt-action-analyze-pythonDenys Vlasenko2010-10-191-48/+43
| | | | | | | | | | | | | | | This patch splits off uuid generation for Python to a separate program: Usage: abrt-action-analyze-python [-vs] -d DIR Calculates and saves UUID and DUPHASH of python crash dumps -v, --verbose be verbose -d DIR Crash dump directory -s Log to syslog Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* bugzilla: trivial cleanupDenys Vlasenko2010-10-191-3/+5
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* change dd_opendir API to return dd pointer (no need to dd_init it separately)Denys Vlasenko2010-10-155-18/+18
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* change dd_create API to return dd pointer (no need to dd_init it separately)Denys Vlasenko2010-10-151-3/+3
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* mve md5 hash from Python plugin to abrtlibDenys Vlasenko2010-10-154-476/+1
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* btparser initial integrationKarel Klic2010-10-142-2/+0
|
* various small fixes to dd usageDenys Vlasenko2010-10-141-1/+1
| | | | | | | | | | | KerneloopsScanner: dd leak on error path in save_oops_to_debug_dump() dump_dir: remove superfluous exist_file_dir check in dd_opendir, dd_create, delete_file_dir; use perror_msg instead of error_msg; correct some sligtly wrong log/error messages; make delete_file_dir return void, since return value is never checked. Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* move uuid generation for CCpp to a separate program (abrt-action-analyze-c)Denys Vlasenko2010-10-141-165/+31
| | | | | | | | | | | | | | | | | | | | | | | | | This patch splits off uuid generation for CCpp to a separate program: Usage: abrt-action-analyze-c [-v] -d DIR Calculates and saves UUID -v, --verbose be verbose -d DIR Crash dump directory Run tested. Thus, now entire cycle of coredump processing/reporting consists of external programs: abrt-action-analyze-c abrt-action-save-package-data abrt-action-install-debuginfo abrt-action-generate-backtrace abrt-action-bugzilla Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* trivial cleanupsDenys Vlasenko2010-10-131-1/+0
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* move bugzilla reporting to a separate program (abrt-action-bugzilla)Denys Vlasenko2010-10-113-938/+91
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch splits off Bugzilla plugin's bugzilla submission generation into a separate tool: abrt-action-bugzilla: invalid option -- 'z' Usage: abrt-action-bugzilla -c CONFFILE -d DIR [-vs] Report a crash to Bugzilla Options: -c FILE Configuration file (may be given many times) -d DIR Crash dump directory -v Verbose -s Log to syslog This change is bigger than previous similar patches: I needed to find a way to pass settings to the child and get back reporting result - all without breaking current interface. I decided to pass configuration via -c CONFFILE: (1) it is trivial to pass /etc/abrt/bugzilla.conf to it, and (2) I can use "-c -" to parse config from stdin. abrtd simply write per-reporting settings to child's stdin. The log and reporting result (success/failure) is taken from child's stdout. After a few more such patches, we can remove XMLRPC and HTTP code and library dependencies from abrtd. Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* preparatory patch with trivial changes onlyDenys Vlasenko2010-10-112-2/+2
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* s/CD_DUPHASH/FILENAME_DUPHASH/gDenys Vlasenko2010-10-113-5/+5
| | | | | | | This is needed to make duplicate hash visible for separate programs looking at crash dumps. Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* Kerneloops*.cpp and dumpoops.cpp uses GListNikola Pajkovsky2010-10-064-19/+26
| | | | | | vector_string_t was replaced by glist in these files Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
* rename abrt-debuginfo-install to abrt-action-install-debuginfoDenys Vlasenko2010-09-231-5/+5
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* add two flags to dd_opendir()Nikola Pajkovsky2010-09-225-43/+9
| | | | | | | | | | | | | | | | | | | DD_CLOSE_ON_OPEN_ERR - free dump_dir structure when opening dump_dir does not exist DD_FAIL_QUIETLY - suppress message when dump directory does not exist VERB1 log(_("Unable to open debug dump '%s'"), pDebugDumpDir); is all removed because there is error_msg("'%s' does not exist", dd->dd_dir); in dd_opendir() which sometimes we don't want to print(DD_FAIL_QUIETLY) example: crash dump directory trimming code running concurrently. Second process may try to delete a directory which is already gone. it should not complain that it is missing. Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com> Acked-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* This patch splits off CCpp's backtrace generation into a separate tool:Denys Vlasenko2010-09-201-221/+37
| | | | | | | | | | | | | | | | | | | | | | | | | | | $ abrt-action-generate-backtrace -z abrt-action-generate-backtrace: invalid option -- 'z' Usage: abrt-action-generate-backtrace -d DIR [-i DIR1:DIR2] [-t SECONDS] [-vs] Generate backtrace, its quality rating, hash, and crashed function Options: -d DIR Crash dump directory -i DIR1:DIR2 Additional debuginfo directories -t SECONDS Kill gdb if it runs for more than SECONDS -v Verbose -s Log to syslog This also allows for debugging and regression testing of abrt-action-generate-backtrace separately - it can be simply run from command-line. Also it provides a better fault isolation - crash in abrt-action-generate-backtrace does not take down abrtd. The code is based on CCpp.cpp. CCpp analyzer is made to spawn a child to do the backtrace generation. Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* unhide dump_dir structure from typedefNikola Pajkovsky2010-09-206-10/+10
| | | | | | no other code changes Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
* CCpp: better handling of uid and exit condition on bt generationDenys Vlasenko2010-09-161-5/+6
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* CCpp: plug a memory leakDenys Vlasenko2010-09-161-0/+1
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* CCpp: skip bt generation earlierDenys Vlasenko2010-09-141-18/+18
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* move sha1 to lib/utils; remove small bit of related C++isms in CCpp.cppDenys Vlasenko2010-09-144-294/+18
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* s/NoSSLVerify/SSLVerify/gNikola Pajkovsky2010-09-141-1/+1
| | | | Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
* CCpp: make gdb timeout configurable; raise it to 120 sec in default config fileDenys Vlasenko2010-09-133-10/+19
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* SQLite3.cpp: get rid of CABRTExceptionNikola Pajkovsky2010-09-101-22/+31
| | | | Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
* fix sigsegvNikola Pajkovsky2010-09-101-3/+11
| | | | | | | copy struct with char* lead to sigsegv. each char* needs to be duplicate. Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
* SQLite3.cpp: replace std::vector by GListNikola Pajkovsky2010-09-083-56/+80
| | | | | Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com> Acked-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* DebugDump.* -> dump_dir.*Nikola Pajkovsky2010-09-0712-12/+0
| | | | | Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com> Acked-off-by: Jiri Moskovcak <jmoskovc@redhat.com>
* DebugDump.cpp: lowercase and rename functionsNikola Pajkovsky2010-09-025-30/+30
| | | | Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
* KerneloopsReporer: throw when missing btNikola Pajkovsky2010-09-021-13/+13
| | | | Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
* added missing dd_close()Jiri Moskovcak2010-09-011-1/+1
|
* RHTSupport.cpp: plug a memory leakKamil Dudka2010-08-311-0/+2
| | | | | Signed-off-by: Kamil Dudka <kdudka@redhat.com> Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* KerneloopsSysLog.cpp: make version extraction code less uglyKamil Dudka2010-08-311-16/+7
| | | | | Signed-off-by: Kamil Dudka <kdudka@redhat.com> Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* KerneloopsReporter.cpp: move misplaced block of codeKamil Dudka2010-08-311-7/+7
| | | | | Signed-off-by: Kamil Dudka <kdudka@redhat.com> Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* FileTransfer.cpp: add forgotten open error testKamil Dudka2010-08-311-0/+4
| | | | | Signed-off-by: Kamil Dudka <kdudka@redhat.com> Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* remove one redundant checkKamil Dudka2010-08-311-32/+28
| | | | | Signed-off-by: Kamil Dudka <kdudka@redhat.com> Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>