]> git.ipfire.org Git - thirdparty/linux.git/commit
cifs: avoid extra calls in posix_info_parse
authorRonnie Sahlberg <lsahlber@redhat.com>
Fri, 18 Jun 2021 00:58:30 +0000 (10:58 +1000)
committerSteve French <stfrench@microsoft.com>
Mon, 21 Jun 2021 02:28:17 +0000 (21:28 -0500)
commitca38fabc31ecf8a2cd14ff92a68936db59106409
tree71e1abb90780b55fc2a264aeedcf618865304a30
parent6efa994e35a402ae4ae2161b6439c94b64816cee
cifs: avoid extra calls in posix_info_parse

In posix_info_parse() we call posix_info_sid_size twice for each of the owner and the group
sid. The first time to check that it is valid, i.e. >= 0 and the second time
to just pass it in as a length to memcpy().
As this is a pure function we know that it can not be negative the second time and this
is technically a false warning in coverity.
However, as it is a pure function we are just wasting cycles by calling it a second time.
Record the length from the first time we call it and save some cycles as well as make
Coverity happy.

Addresses-Coverity-ID: 1491379 ("Argument can not be negative")

Signed-off-by: Ronnie Sahlberg <lsahlber@redhat.com>
Signed-off-by: Steve French <stfrench@microsoft.com>
fs/cifs/smb2pdu.c