svn commit: r329296 - head/sbin/devd

Eitan Adler eadler at FreeBSD.org
Thu Feb 15 03:22:53 UTC 2018


Author: eadler
Date: Thu Feb 15 03:22:53 2018
New Revision: 329296
URL: https://svnweb.freebsd.org/changeset/base/329296

Log:
  devd: don't pass &fds in useless parameters to select(2)
  
  select(2) should be declared as restrict. In addition the only fd in
  the fdset is open O_RDONLY, and it's not a socket that can provide OOB
  notifications,
  
  Reviewed by:	ian, imp, vangyzen

Modified:
  head/sbin/devd/devd.cc

Modified: head/sbin/devd/devd.cc
==============================================================================
--- head/sbin/devd/devd.cc	Thu Feb 15 03:22:04 2018	(r329295)
+++ head/sbin/devd/devd.cc	Thu Feb 15 03:22:53 2018	(r329296)
@@ -1021,7 +1021,7 @@ event_loop(void)
 			tv.tv_usec = 0;
 			FD_ZERO(&fds);
 			FD_SET(fd, &fds);
-			rv = select(fd + 1, &fds, &fds, &fds, &tv);
+			rv = select(fd + 1, &fds, NULL, NULL, &tv);
 			// No events -> we've processed all pending events
 			if (rv == 0) {
 				devdlog(LOG_DEBUG, "Calling daemon\n");


More information about the svn-src-all mailing list