diff options
author | Ronnie Sahlberg <lsahlber@redhat.com> | 2021-06-18 10:58:30 +1000 |
---|---|---|
committer | Steve French <stfrench@microsoft.com> | 2021-06-20 21:28:17 -0500 |
commit | ca38fabc31ecf8a2cd14ff92a68936db59106409 (patch) | |
tree | 71e1abb90780b55fc2a264aeedcf618865304a30 /tools/perf/scripts/python/exported-sql-viewer.py | |
parent | 6efa994e35a402ae4ae2161b6439c94b64816cee (diff) |
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>
Diffstat (limited to 'tools/perf/scripts/python/exported-sql-viewer.py')
0 files changed, 0 insertions, 0 deletions