> In the original code we check if (servl == NULL) twice. ÂThe first time > should print the message that cfmuxl_remove_uplayer() failed and set > "ret" correctly, but instead it just returns success. ÂThe second check > should be checking the value of "ret" instead of "servl". > > Signed-off-by: Dan Carpenter <error27@xxxxxxxxx> Thank you for spotting and correcting this. Looks good to me (reviewed only) Acked-by: Sjur Braendeland <sjur.brandeland@xxxxxxxxxxxxxx> -- 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
