summaryrefslogtreecommitdiffstats
path: root/src
Commit message (Collapse)AuthorAgeFilesLines
...
* introduce and use xmalloc_fgets/fgetlineDenys Vlasenko2010-10-222-21/+23
| | | | | | | 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>
* restore correct handling of argv[1] of scriptsDenys Vlasenko2010-10-221-24/+21
| | | | | | Bad boy Karel "improved" it incorrectly, and did not revert it. Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* dump_dir: make open fail if there is no "analyzer" file.Denys Vlasenko2010-10-211-18/+12
| | | | | | | | | Also adjusts abrt-action-print to return better error message: '/tmp' is not a crash dump directory 1 Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* GUI: make the "install debuginfo" hint selectable rhbz#644343Jiri Moskovcak2010-10-211-0/+1
|
* s/dst++/*dst++/ in previous commitDenys Vlasenko2010-10-211-1/+1
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* abrt-action-rhtsupport: fix error handlingDenys Vlasenko2010-10-211-2/+29
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* move logger reporting to a separate program (abrt-action-print)Denys Vlasenko2010-10-202-1/+105
| | | | | | | | | | | | | | | | | 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-202-1/+335
| | | | | | | | | | | | | | | | 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>
* GUI: minor fix to dbus backend to prevent exceptions when kerneloops happen ↵Jiri Moskovcak2010-10-191-2/+1
| | | | | | | | while gui is open - if oops happen abrtd sends a crash signal without uid_str parameter, but the old code expects 3 paramaters and throws an exception this patch fixes this problem by making the uid parameter optional
* move uuid/duphash generation for Kerneloops to abrt-action-analyze-oopsDenys Vlasenko2010-10-192-0/+195
| | | | | | | | | | | | 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-192-0/+138
| | | | | | | | | | | | | | | 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-1/+0
| | | | 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-27/+27
| | | | 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-152-6/+4
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* abrtd: fix "unset PATH" problem when started from dbus daemonDenys Vlasenko2010-10-151-0/+7
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* abrt-action-analyze-c: update --help descriptionDenys Vlasenko2010-10-151-1/+1
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* btparser integration: use ABRT's strbuf instead of internal oneKarel Klic2010-10-148-340/+65
|
* Merge branch 'master' of ssh://git.fedorahosted.org/git/abrt into retraceKarel Klic2010-10-141-1/+1
|\
| * fix my embarrassing error (check for equality is strncmp == 0!)Denys Vlasenko2010-10-141-1/+1
| | | | | | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* | btparser integration: merge it into ABRT's directory hierarchyKarel Klic2010-10-1425-5/+4549
| |
* | btparser integration: sync with recent UUID changeKarel Klic2010-10-141-2/+2
| |
* | btparser initial integrationKarel Klic2010-10-146-493/+67
|/
* move uuid generation for CCpp to a separate program (abrt-action-analyze-c)Denys Vlasenko2010-10-143-2/+259
| | | | | | | | | | | | | | | | | | | | | | | | | 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>
* whitespace fixDenys Vlasenko2010-10-141-1/+1
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* better name for abrt-FOO coredumpsDenys Vlasenko2010-10-131-2/+3
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* don't catch crashes of any program which starts with "abrt", not only abrtdDenys Vlasenko2010-10-131-3/+3
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* restore abrt-action-save-package-data's description in --helpDenys Vlasenko2010-10-131-4/+9
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* trivial cleanupsDenys Vlasenko2010-10-131-4/+4
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* Add a warning comment that options and their enums should be in syncDenys Vlasenko2010-10-124-18/+11
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* GUI: added search box to backtrace view rhbz#612017Jiri Moskovcak2010-10-121-1/+143
|
* Return a bitmask of found options from parse_opts()Denys Vlasenko2010-10-124-50/+49
| | | | | | | | | | | | | This patch makes it easier to analyze "bool" type options - now they are all just bits in a single return value, no need to have an integer variable for each option. The previous behavior is retained too: it is useful for "cumulative" options like -vvv. The patch also removes all OPT_GROUPs. Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* move bugzilla reporting to a separate program (abrt-action-bugzilla)Denys Vlasenko2010-10-112-190/+130
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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>
* move load_crash_data_from_debug_dump from MiddleWare to CrashTypesDenys Vlasenko2010-10-111-123/+0
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* preparatory patch with trivial changes onlyDenys Vlasenko2010-10-111-5/+5
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* adding abrt-action-bugzilla.cpp as an exact copy of Bugzilla.cppDenys Vlasenko2010-10-111-0/+989
| | | | | | | | The mext commit will be much more readable: insted of huge insert, you will actually see that there are very few changes to actual bz code in that file. Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* s/CD_DUPHASH/FILENAME_DUPHASH/gDenys Vlasenko2010-10-114-16/+20
| | | | | | | This is needed to make duplicate hash visible for separate programs looking at crash dumps. Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* abrt-action-generate-backtrace.c: fix -i optionNikola Pajkovsky2010-10-081-1/+1
| | | | Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
* trivial: remove unused option_index'sDenys Vlasenko2010-10-081-2/+1
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* Remove arrays in usage text, use a single string insteadDenys Vlasenko2010-10-084-16/+4
| | | | | | | | | | | | | | | | | | | | | | | | Currently, usage is an array of pointers to string. It is meant to support this: usage: foo --bar or: foo --baz or: foo -z The patch removes this and makes usage just a string: usage: foo --bar Why remove this? (1) We can achieve old format by using the usage string with many lines: usage = "foo --bar\n" " or: foo --baz\n" " or: foo -z"; (2) We don't use multi-line usage anyway. Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* add --help option in parse_opts(), not at every call site.Denys Vlasenko2010-10-084-38/+26
| | | | | | | This patch removes the need to add --help option to every program. It is added (and handled) by parse_opts(). Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* use OPT_GROUP macro for help optionNikola Pajkovsky2010-10-084-0/+4
| | | | Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
* use OPT__VERBOSE macro for help optionNikola Pajkovsky2010-10-084-4/+4
| | | | Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
* use OPT__HELP macro for help optionNikola Pajkovsky2010-10-084-4/+4
| | | | Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
* Merge branch 'unify-parser'Nikola Pajkovsky2010-10-084-121/+84
|\ | | | | | | | | | | | | | | * unify-parser: abrt-server uses new parser abrt-action-save-package-data uses new parser use a new INTEGER option abrt-action-generate-backtrace uses new parser
| * abrt-server uses new parserNikola Pajkovsky2010-10-071-30/+23
| | | | | | | | Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
| * abrt-action-save-package-data uses new parserNikola Pajkovsky2010-10-071-33/+26
| | | | | | | | Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
| * use a new INTEGER optionNikola Pajkovsky2010-10-071-11/+1
| | | | | | | | Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
| * abrt-action-generate-backtrace uses new parserNikola Pajkovsky2010-10-071-47/+34
| | | | | | | | Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
* | add a bit more logging in places. fix typosDenys Vlasenko2010-10-073-4/+6
|/ | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* Settings.cpp: avoid double free corruptionNikola Pajkovsky2010-10-061-0/+5
| | | | | | | | | | | | I meant, that settings_free() should not only do free(g_settings_*); but also should do g_settings_* = NULL; Otherwise calling settings_free() twice will cause double-free and likely segfault. Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>