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

Jakub Filak jfilak at redhat.com
Mon Jun 10 14:30:17 UTC 2013


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




More information about the Crash-catcher mailing list