media: usb: go7007: s2250-board: fix leak in probe()
Call i2c_unregister_device(audio) on this error path.
Fixes: d3b2ccd9e3
("[media] s2250: convert to the control framework")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
This commit is contained in:
Родитель
7c38a551bd
Коммит
67e4550ecd
|
@ -504,6 +504,7 @@ static int s2250_probe(struct i2c_client *client,
|
|||
u8 *data;
|
||||
struct go7007 *go = i2c_get_adapdata(adapter);
|
||||
struct go7007_usb *usb = go->hpi_context;
|
||||
int err = -EIO;
|
||||
|
||||
audio = i2c_new_dummy_device(adapter, TLV320_ADDRESS >> 1);
|
||||
if (IS_ERR(audio))
|
||||
|
@ -532,11 +533,8 @@ static int s2250_probe(struct i2c_client *client,
|
|||
V4L2_CID_HUE, -512, 511, 1, 0);
|
||||
sd->ctrl_handler = &state->hdl;
|
||||
if (state->hdl.error) {
|
||||
int err = state->hdl.error;
|
||||
|
||||
v4l2_ctrl_handler_free(&state->hdl);
|
||||
kfree(state);
|
||||
return err;
|
||||
err = state->hdl.error;
|
||||
goto fail;
|
||||
}
|
||||
|
||||
state->std = V4L2_STD_NTSC;
|
||||
|
@ -600,7 +598,7 @@ fail:
|
|||
i2c_unregister_device(audio);
|
||||
v4l2_ctrl_handler_free(&state->hdl);
|
||||
kfree(state);
|
||||
return -EIO;
|
||||
return err;
|
||||
}
|
||||
|
||||
static int s2250_remove(struct i2c_client *client)
|
||||
|
|
Загрузка…
Ссылка в новой задаче