Blackfin: sram_free_with_lsl: do not ignore return value of sram_free
If there was an error in the lower free functions, we need to pass that back up so the calling process is able to check things. Signed-off-by: Mike Frysinger <vapier@gentoo.org>
This commit is contained in:
parent
a8b1988609
commit
25f3ff2c44
|
@ -704,18 +704,18 @@ int sram_free_with_lsl(const void *addr)
|
|||
{
|
||||
struct sram_list_struct *lsl, **tmp;
|
||||
struct mm_struct *mm = current->mm;
|
||||
int ret = -1;
|
||||
|
||||
for (tmp = &mm->context.sram_list; *tmp; tmp = &(*tmp)->next)
|
||||
if ((*tmp)->addr == addr)
|
||||
goto found;
|
||||
return -1;
|
||||
found:
|
||||
lsl = *tmp;
|
||||
sram_free(addr);
|
||||
*tmp = lsl->next;
|
||||
kfree(lsl);
|
||||
if ((*tmp)->addr == addr) {
|
||||
lsl = *tmp;
|
||||
ret = sram_free(addr);
|
||||
*tmp = lsl->next;
|
||||
kfree(lsl);
|
||||
break;
|
||||
}
|
||||
|
||||
return 0;
|
||||
return ret;
|
||||
}
|
||||
EXPORT_SYMBOL(sram_free_with_lsl);
|
||||
|
||||
|
|
Loading…
Reference in a new issue