bigocean: Release reference before returning
Fix a bug where instance references were not being released before
returning from IOCTL.
bug: 312438904
Change-Id: I55f16e9856aa50e8312b444cb384bebcf7b41174
(cherry picked from commit d912d091a31cfe27cc74ea43ed6be88d7b585900)
Signed-off-by: Vinay Kalia <vinaykalia@google.com>
diff --git a/bigo.c b/bigo.c
index 9927798..6a35ae3 100644
--- a/bigo.c
+++ b/bigo.c
@@ -394,7 +394,8 @@
if (copy_regs_from_user(core, &desc, user_desc, job)) {
pr_err("Failed to copy regs from user\n");
- return -EFAULT;
+ rc = -EFAULT;
+ break;
}
hbd = (((u32*)job->regs)[3]) & BIGO_HBD_BIT;
@@ -406,7 +407,8 @@
if(enqueue_prioq(core, inst)) {
pr_err("Failed enqueue frame\n");
- return -EFAULT;
+ rc = -EFAULT;
+ break;
}
ret = wait_for_completion_timeout(
@@ -433,7 +435,8 @@
case BIGO_IOCX_MAP:
if (copy_from_user(&mapping, user_desc, sizeof(mapping))) {
pr_err("Failed to copy from user\n");
- return -EFAULT;
+ rc = -EFAULT;
+ break;
}
rc = bigo_map(core, inst, &mapping);
if (rc)
@@ -446,7 +449,8 @@
case BIGO_IOCX_UNMAP:
if (copy_from_user(&mapping, user_desc, sizeof(mapping))) {
pr_err("Failed to copy from user\n");
- return -EFAULT;
+ rc = -EFAULT;
+ break;
}
rc = bigo_unmap(inst, &mapping);
if (rc)
@@ -461,7 +465,8 @@
case BIGO_IOCX_CONFIG_FRMSIZE:
if (copy_from_user(&frmsize, user_desc, sizeof(frmsize))) {
pr_err("Failed to copy from user\n");
- return -EFAULT;
+ rc = -EFAULT;
+ break;
}
bigo_config_frmsize(inst, &frmsize);
break;