Author: kp
Date: Fri Apr 26 13:00:25 2019
New Revision: 346742
URL: https://svnweb.freebsd.org/changeset/base/346742

Log:
  MFC r346349:
  
  pf: No need to M_NOWAIT in DIOCRSETTFLAGS
  
  Now that we don't hold a lock during DIOCRSETTFLAGS memory allocation we can
  use M_WAITOK.
  
  Pointed out by:       glebius@

Modified:
  stable/11/sys/netpfil/pf/pf_ioctl.c
Directory Properties:
  stable/11/   (props changed)

Modified: stable/11/sys/netpfil/pf/pf_ioctl.c
==============================================================================
--- stable/11/sys/netpfil/pf/pf_ioctl.c Fri Apr 26 13:00:22 2019        
(r346741)
+++ stable/11/sys/netpfil/pf/pf_ioctl.c Fri Apr 26 13:00:25 2019        
(r346742)
@@ -2693,11 +2693,7 @@ DIOCCHANGEADDR_error:
 
                totlen = io->pfrio_size * sizeof(struct pfr_table);
                pfrts = mallocarray(io->pfrio_size, sizeof(struct pfr_table),
-                   M_TEMP, M_NOWAIT);
-               if (pfrts == NULL) {
-                       error = ENOMEM;
-                       break;
-               }
+                   M_TEMP, M_WAITOK);
                error = copyin(io->pfrio_buffer, pfrts, totlen);
                if (error) {
                        free(pfrts, M_TEMP);
_______________________________________________
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"

Reply via email to