lkml.org 
[lkml]   [2014]   [Mar]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH 4/7] staging: cxt1e1: fix checkpatch error 'assignment in if condition'
From
Yes, it need to add brace in inner loop.
I will send again.

Thanks for review.
Daeseok Youn

2014-03-04 18:04 GMT+09:00, Dan Carpenter <dan.carpenter@oracle.com>:
> On Tue, Mar 04, 2014 at 11:09:39AM +0900, Daeseok Youn wrote:
>> @@ -1174,7 +1179,8 @@ cleanup_hdlc(void)
>> ci = (ci_t *)(netdev_priv(hi->ndev));
>> for (j = 0; j < ci->max_port; j++)
>> for (k = 0; k < MUSYCC_NCHANS; k++)
>> - if ((ndev = ci->port[j].chan[k]->user))
>> + ndev = ci->port[j].chan[k]->user;
>> + if (ndev)
>> {
>> do_deluser(ndev, 0);
>> }
>
>
> This patch introduces a bug here because the inner for loop now has two
> statement but no curly braces.
>
> regards,
> dan carpenter
>


\
 
 \ /
  Last update: 2014-03-04 12:01    [W:0.117 / U:0.076 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site