| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
| |
Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
| |
Transmitted-via: Transifex (translate.fedoraproject.org)
|
|
|
|
| |
Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
|
|
|
|
| |
Transmitted-via: Transifex (translate.fedoraproject.org)
|
|
|
|
| |
Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
|
|
|
|
| |
Transmitted-via: Transifex (translate.fedoraproject.org)
|
|
|
|
| |
Transmitted-via: Transifex (translate.fedoraproject.org)
|
|
|
|
| |
Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
|
|
|
|
| |
Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
|
|
|
|
| |
Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
|
|
|
|
| |
Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
|
|
|
|
| |
Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
| |
Transmitted-via: Transifex (translate.fedoraproject.org)
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
| |
Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
|
|
|
|
| |
Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
| |
This is needed to make duplicate hash visible for separate programs
looking at crash dumps.
Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
|
|
|
|
| |
Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
|
|
|
|
| |
Transmitted-via: Transifex (translate.fedoraproject.org)
|
|
|
|
| |
Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
| |
Transmitted-via: Transifex (translate.fedoraproject.org)
|
|
|
|
|
|
|
| |
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>
|
|
|
|
| |
Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
|
|
|
|
| |
Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
|
|
|
|
| |
Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
|
|\
| |
| |
| |
| |
| |
| |
| | |
* 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
|
| |
| |
| |
| | |
Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
|
| |
| |
| |
| | |
Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
|
| |
| |
| |
| | |
Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
|
| |
| |
| |
| | |
Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
|
| |
| |
| |
| | |
Transmitted-via: Transifex (translate.fedoraproject.org)
|
| |
| |
| |
| | |
Transmitted-via: Transifex (translate.fedoraproject.org)
|
| |
| |
| |
| | |
Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
|
| |
| |
| |
| | |
Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
|
|/
|
|
| |
Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
|
|
|
|
| |
Transmitted-via: Transifex (translate.fedoraproject.org)
|
|
|
|
| |
Transmitted-via: Transifex (translate.fedoraproject.org)
|
|
|
|
| |
Transmitted-via: Transifex (translate.fedoraproject.org)
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|\
| |
| |
| |
| |
| |
| | |
* vector_string_t:
report.cpp: split() uses GList
Kerneloops*.cpp and dumpoops.cpp uses GList
Daemon.cpp: FindNewDumps() uses GList
|
| |
| |
| |
| | |
Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
|
| |
| |
| |
| |
| |
| | |
vector_string_t was replaced by glist in these files
Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
|
| |
| |
| |
| |
| |
| | |
transform vector_string_t to GList
Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
|