[ABRT PATCH] a-d-oops: obtain kernel version from the oops

[Martin Milata]

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 
---
 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");

[Jakub Filak]

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
>
> 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");