Don't test boolean return against != 1.

Don't needlessly assign the error variable in an if statement.

1 files changed, 4 insertions, 4 deletions

diff --git a/sys/pci/pci.c b/sys/pci/pci.c
index a78903e..9a756a8 100644
--- a/sys/pci/pci.c
+++ b/sys/pci/pci.c
@@ -662,8 +662,8 @@ pci_ioctl(dev_t dev, u_long cmd, caddr_t data, int flag, struct proc *p)
 			/*
 			 * Check the user's buffer to make sure it's readable.
 			 */
-			if ((error = useracc((caddr_t)cio->patterns,
-			                     cio->pat_buf_len, B_READ)) != 1){
+			if (!useracc((caddr_t)cio->patterns,
+				    cio->pat_buf_len, B_READ)) {
 				printf("pci_ioctl: pattern buffer %p, "
 				       "length %u isn't user accessible for"
 				       " READ\n", cio->patterns,
@@ -698,8 +698,8 @@ pci_ioctl(dev_t dev, u_long cmd, caddr_t data, int flag, struct proc *p)
 		/*
 		 * Make sure we can write to the match buffer.
 		 */
-		if ((error = useracc((caddr_t)cio->matches, cio->match_buf_len,
-				     B_WRITE)) != 1) {
+		if (!useracc((caddr_t)cio->matches,
+			     cio->match_buf_len, B_WRITE)) {
 			printf("pci_ioctl: match buffer %p, length %u "
 			       "isn't user accessible for WRITE\n",
 			       cio->matches, cio->match_buf_len);