Am 08.04.2015 11:51, schrieb Julia Lawall: >> void ext4_fname_crypto_free_buffer(void **buf) >> { >> - if (*buf == NULL || IS_ERR(buf)) >> + if (*buf == NULL || IS_ERR(*buf)) > > Why not use IS_ERR_OR_NULL? > > julia why test *buf == NULL ? xfree() can handle this. the question is do programm depend on *buf=NULL. In case of IS_ERR(*buf) *buf will be left unchanged and later prgramms may things there is a buffer available ? > >> return; >> kfree(*buf); >> *buf = NULL; >> -- >> 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 >> > -- > 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 > -- 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