Print this page
6659 nvlist_free(NULL) is a no-op

Split Close
Expand all
Collapse all
          --- old/usr/src/lib/cfgadm_plugins/ib/common/cfga_ib.c
          +++ new/usr/src/lib/cfgadm_plugins/ib/common/cfga_ib.c
↓ open down ↓ 1962 lines elided ↑ open up ↑
1963 1963                                  ++count;
1964 1964                          }
1965 1965  
1966 1966                          clp->ap_class[0] = '\0'; /* Filled by libcfgadm */
1967 1967                          clp->ap_busy = 0;
1968 1968                          clp->ap_status_time = (time_t)-1;
1969 1969                  } /* end of while */
1970 1970          }
1971 1971  
1972 1972          S_FREE(snap_data);
1973      -        if (nvl)
1974      -                nvlist_free(nvl);
     1973 +        nvlist_free(nvl);
1975 1974  
1976 1975          /*
1977 1976           * if a cmdline specified ap_id doesn't match the known list of ap_ids
1978 1977           * then report an error right away
1979 1978           */
1980 1979          rv = (apid_matched ==  B_TRUE) ? CFGA_IB_OK : CFGA_IB_AP_ERR;
1981 1980          return (ib_err_msg(errstring, rv, ap_id, errno));
1982 1981  }
1983 1982  
1984 1983  
↓ open down ↓ 144 lines elided ↑ open up ↑
XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX