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

Split Close
Expand all
Collapse all
          --- old/usr/src/lib/cfgadm_plugins/sbd/common/ap_rcm.c
          +++ new/usr/src/lib/cfgadm_plugins/sbd/common/ap_rcm.c
↓ open down ↓ 789 lines elided ↑ open up ↑
 790  790  
 791  791          if (cmd == CMD_RCM_CAP_DEL) {
 792  792                  rv = (*rcm->request_capacity_change)(hd, "SUNW_cpu",
 793  793                      flags, nvl, rinfo);
 794  794          } else {
 795  795                  rv = (*rcm->notify_capacity_change)(hd, "SUNW_cpu",
 796  796                      flags & ~RCM_FORCE, nvl, rinfo);
 797  797          }
 798  798  
 799  799  done:
 800      -        if (nvl)
 801      -                nvlist_free(nvl);
      800 +        nvlist_free(nvl);
 802  801          s_free(oldcpuids);
 803  802          s_free(newcpuids);
 804  803          return (rv);
 805  804  }
 806  805  
 807  806  static int
 808  807  ap_rcm_cap_mem(apd_t *a, rcmd_t *rcm, rcm_handle_t *hd, uint_t flags,
 809  808          rcm_info_t **rinfo, int cmd, long change)
 810  809  {
 811  810          int rv;
↓ open down ↓ 642 lines elided ↑ open up ↑
XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX