On Thu, May 21, 2015 at 03:21:56PM +0200, Jan Kara wrote: > On Wed 20-05-15 11:13:15, Dan Carpenter wrote: > > There are some missing braces here which means this function never > > succeeds. > > > > Fixes: e9d4cf411f75 ('udf: improve error management in udf_CS0toUTF8()') > > Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> > Thanks for the patch. Added to my tree. Actually, the missing parenthesis > don't cause any real issues since almost all volumes have volIdent and thus > ret gets set to 0 in the previous if. But it's certainly good to fix. Ah... I didn't read carefully. Thanks. regards, dan carpenter -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html