The FILENAME_KERNEL was overwritten by dd_create_basic_files -- this can be worked around by calling the function first and saving the version later.
Signed-off-by: Martin Milata mmilata@redhat.com --- src/plugins/abrt-dump-oops.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/plugins/abrt-dump-oops.c b/src/plugins/abrt-dump-oops.c index 4ae725c..83ae5a7 100644 --- a/src/plugins/abrt-dump-oops.c +++ b/src/plugins/abrt-dump-oops.c @@ -198,8 +198,8 @@ static unsigned create_oops_dump_dirs(GList *oops_list, unsigned oops_cnt)
if (dd) { - save_oops_data_in_dump_dir(dd, (char*)g_list_nth_data(oops_list, idx++), proc_modules); dd_create_basic_files(dd, /*uid:*/ my_euid, NULL); + save_oops_data_in_dump_dir(dd, (char*)g_list_nth_data(oops_list, idx++), proc_modules); dd_save_text(dd, FILENAME_ABRT_VERSION, VERSION); dd_save_text(dd, FILENAME_ANALYZER, "Kerneloops"); dd_save_text(dd, FILENAME_TYPE, "Kerneloops");
Pushed! Thank you!
On Mon, 2013-06-10 at 14:57 +0200, Martin Milata wrote:
The FILENAME_KERNEL was overwritten by dd_create_basic_files -- this can be worked around by calling the function first and saving the version later.
Signed-off-by: Martin Milata mmilata@redhat.com
src/plugins/abrt-dump-oops.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/plugins/abrt-dump-oops.c b/src/plugins/abrt-dump-oops.c index 4ae725c..83ae5a7 100644 --- a/src/plugins/abrt-dump-oops.c +++ b/src/plugins/abrt-dump-oops.c @@ -198,8 +198,8 @@ static unsigned create_oops_dump_dirs(GList *oops_list, unsigned oops_cnt)
if (dd) {
save_oops_data_in_dump_dir(dd, (char*)g_list_nth_data(oops_list, idx++), proc_modules); dd_create_basic_files(dd, /*uid:*/ my_euid, NULL);
save_oops_data_in_dump_dir(dd, (char*)g_list_nth_data(oops_list, idx++), proc_modules); dd_save_text(dd, FILENAME_ABRT_VERSION, VERSION); dd_save_text(dd, FILENAME_ANALYZER, "Kerneloops"); dd_save_text(dd, FILENAME_TYPE, "Kerneloops");
crash-catcher@lists.fedorahosted.org