From: Yannick Brosseau Date: Sat, 3 Dec 2011 02:13:32 +0000 (-0500) Subject: staging: lttng: Fix recent modifications to string_from_user operation X-Git-Tag: v3.3-rc1~152^2~101^2~12 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=643167423858052c26b2dfaf332c1ec0b472ea7a;p=profile%2Fivi%2Fkernel-adaptation-intel-automotive.git staging: lttng: Fix recent modifications to string_from_user operation Include: a fix for a recently introduced change: obviously max_t should be used instead of min_t here. Also, a likely should apply to the result of the comparison, not the variable per se. Signed-off-by: Yannick Brosseau Signed-off-by: Mathieu Desnoyers Signed-off-by: Greg Kroah-Hartman --- diff --git a/drivers/staging/lttng/probes/lttng-events.h b/drivers/staging/lttng/probes/lttng-events.h index ff6273f..d486994 100644 --- a/drivers/staging/lttng/probes/lttng-events.h +++ b/drivers/staging/lttng/probes/lttng-events.h @@ -347,7 +347,7 @@ static __used struct lttng_probe_desc TP_ID(__probe_desc___, TRACE_SYSTEM) = { #undef __string_from_user #define __string_from_user(_item, _src) \ __event_len += __dynamic_len[__dynamic_len_idx++] = \ - min_t(size_t, strlen_user(_src), 1); + max_t(size_t, strlen_user(_src), 1); #undef TP_PROTO #define TP_PROTO(args...) args @@ -557,7 +557,7 @@ __assign_##dest##_2: \ (void) __typemap.dest; \ lib_ring_buffer_align_ctx(&__ctx, ltt_alignof(__typemap.dest));\ __ustrlen = __get_dynamic_array_len(dest); \ - if (likely(__ustrlen) > 1) { \ + if (likely(__ustrlen > 1)) { \ __chan->ops->event_write_from_user(&__ctx, src, \ __ustrlen - 1); \ } \