Am Montag, dem 27.03.2023 um 11:58 +0000 schrieb Wei Chen: > hi846_init_controls doesn't clean the allocated ctrl_hdlr > in case there is a failure, which causes memleak. Add > v4l2_ctrl_handler_free to free the resource properly. > > Fixes: e8c0882685f9 ("media: i2c: add driver for the SK Hynix Hi-846 > 8M pixel camera") > Signed-off-by: Wei Chen <harperchen1110@xxxxxxxxx> just for the future: you don't need to send a new revision for tags- addition only. That's usually something that maintainers can do when applying, also (semi-)automated. But *if* you send a new revision (due to other changes), you *should* add all appropriate tags. So for this new revision you forgot this: Reviewed-by: Martin Kepplinger <martin.kepplinger@xxxxxxx> but, as you now know, you should not create a new revision for this alone. thanks, martin