[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