It was a very minor fix that an if statement had curly braces with just single line of code inside. Signed-off-by: Sangho Yi <antiroot@xxxxxxxxx> --- drivers/staging/csr/data_tx.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/csr/data_tx.c b/drivers/staging/csr/data_tx.c index 8af0358..1bea7f0 100644 --- a/drivers/staging/csr/data_tx.c +++ b/drivers/staging/csr/data_tx.c @@ -20,9 +20,8 @@ int uf_verify_m4 (unifi_priv_t *priv, const unsigned char *packet, const unsigned char *p = packet; u16 keyinfo; - if (length < (4 + 5 + 8 + 32 + 16 + 8 + 8 + 16 + 1 + 8)) { + if (length < (4 + 5 + 8 + 32 + 16 + 8 + 8 + 16 + 1 + 8)) return 1; - } p += 8; keyinfo = p[5] << 8 | p[6]; /* big-endian */ -- 1.7.9.5 _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/devel