diff options
author | David Herrmann <dh.herrmann@googlemail.com> | 2012-03-28 11:48:42 +0200 |
---|---|---|
committer | Gustavo Padovan <gustavo@padovan.org> | 2012-05-09 00:41:36 -0300 |
commit | 0dea0141a9d9c33a669c546f8f5d9f44a0557375 (patch) | |
tree | fdee2cb96672cbd583738a13086210dc528d1d1d /security | |
parent | 053262dce5a0cc503456d15e204cde205dcf927e (diff) | |
download | op-kernel-dev-0dea0141a9d9c33a669c546f8f5d9f44a0557375.zip op-kernel-dev-0dea0141a9d9c33a669c546f8f5d9f44a0557375.tar.gz |
Bluetooth: vhci: Ignore return code of nonseekable_open()
The comment in ./fs/open.c clearly states that nonseekable_open() will
never fail. Therefore, we can safely ignore the return code. This is the
recommended way to deal with nonseekable_open().
Our current code looks like nonseekable_open() is checked for the return
code. However, if we check the return code, we must also kfree() our
private data if the open fails. To avoid this overhead and to avoid
confusion, we simply drop the return code and return 0.
Signed-off-by: David Herrmann <dh.herrmann@googlemail.com>
Acked-by: Marcel Holtmann <marcel@holtmann.org>
Signed-off-by: Johan Hedberg <johan.hedberg@intel.com>
Diffstat (limited to 'security')
0 files changed, 0 insertions, 0 deletions