Code review comments from jeffpc

*** 2619,2634 ****
                  { 0, MSG_FORBIDNULLMAP },
                  { 0, MSG_NOEXECSTACK },
                  { 0, 0 }
          };
          static CONV_EXPN_FIELD_ARG conv_arg = {
!             NULL, sizeof (secflags_buf->buf) };
          int i;
  
!         for (i = 0; vda[i].v_msg != 0; i++) {
                  vda[i].v_val = secflag_to_bit(i);
-         }
  
          if (flags == 0)
                  return (MSG_ORIG(MSG_GBL_ZERO));
  
          conv_arg.buf = secflags_buf->buf;
--- 2619,2634 ----
                  { 0, MSG_FORBIDNULLMAP },
                  { 0, MSG_NOEXECSTACK },
                  { 0, 0 }
          };
          static CONV_EXPN_FIELD_ARG conv_arg = {
!             NULL, sizeof (secflags_buf->buf)
!         };
          int i;
  
!         for (i = 0; vda[i].v_msg != 0; i++)
                  vda[i].v_val = secflag_to_bit(i);
  
          if (flags == 0)
                  return (MSG_ORIG(MSG_GBL_ZERO));
  
          conv_arg.buf = secflags_buf->buf;