[SATYR PATCH] Don't include user info in kerneloops ureport

Richard Marko rmarko at redhat.com
Wed May 22 09:52:24 UTC 2013


Pushed.

On 05/21/2013 04:30 PM, Martin Milata wrote:
> Closes #80.
>
> Signed-off-by: Martin Milata <mmilata at redhat.com>
> ---
>  lib/report.c | 15 +++++++++------
>  1 file changed, 9 insertions(+), 6 deletions(-)
>
> diff --git a/lib/report.c b/lib/report.c
> index 96ef1d5..fae62f2 100644
> --- a/lib/report.c
> +++ b/lib/report.c
> @@ -105,12 +105,15 @@ problem_object_string(struct sr_report *report, const char *report_type)
>          sr_strbuf_append_str(strbuf, "\n");
>      }
>  
> -    /* User type. */
> -    sr_strbuf_append_strf(strbuf, ",   \"user\": {   \"root\": %s\n"  \
> -                                  "            ,   \"local\": %s\n" \
> -                                  "            }\n",
> -                          report->user_root ? "true" : "false",
> -                          report->user_local ? "true" : "false");
> +    if (report->report_type != SR_REPORT_KERNELOOPS)
> +    {
> +        /* User type (not applicable to koopses). */
> +        sr_strbuf_append_strf(strbuf, ",   \"user\": {   \"root\": %s\n"  \
> +                                      "            ,   \"local\": %s\n" \
> +                                      "            }\n",
> +                              report->user_root ? "true" : "false",
> +                              report->user_local ? "true" : "false");
> +    }
>  
>      /* Stacktrace. */
>  


-- 
Richard Marko



More information about the Crash-catcher mailing list