提交 d734f92b 编写于 作者: A Adrian Bunk 提交者: Jody McIntyre

drivers/ieee1394/raw1394.c: fix a NULL pointer

The coverity checker spotted that this was a NULL pointer dereference in
the "if (copy_from_user(...))" case since the next step is to
kfree(cache->filled_head).

There's no need to free cache at this point, and it's getting free'd
later.
Signed-off-by: NAdrian Bunk <bunk@stusta.de>
Signed-off-by: NJody McIntyre <scjody@modernduck.com>
上级 977545e3
......@@ -2131,7 +2131,6 @@ static int modify_config_rom(struct file_info *fi, struct pending_request *req)
req->req.length)) {
csr1212_release_keyval(fi->csr1212_dirs[dr]);
fi->csr1212_dirs[dr] = NULL;
CSR1212_FREE(cache);
ret = -EFAULT;
} else {
cache->len = req->req.length;
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册