d/freebsd-nq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Remove redundant check for "(dinfo != NULL)", it has already been performed

Browse Source 
as the first part of this complex loop conditional.

Found by:    PVS Static Analysis
This commit is contained in:
Stefan Eßer 2016-02-19 16:43:03 +00:00
parent 308c3c240f
commit 755ae20cd7
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
sys/dev/pci/pci_user.c
View File

@ -709,9 +709,9 @@ pci_ioctl(struct cdev *dev, u_long cmd, caddr_t data, int flag, struct thread *t
* that match the user's criteria.
*/
for (cio->num_matches = 0, error = 0, i = 0,
dinfo = STAILQ_FIRST(devlist_head);
(dinfo != NULL) && (cio->num_matches < ionum)
&& (error == 0) && (i < pci_numdevs) && (dinfo != NULL);
dinfo = STAILQ_FIRST(devlist_head);
(dinfo != NULL) && (cio->num_matches < ionum) &&
(error == 0) && (i < pci_numdevs));
dinfo = STAILQ_NEXT(dinfo, pci_links), i++) {
if (i < cio->offset)