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

Split Close
Expand all
Collapse all
          --- old/usr/src/uts/common/contract/device.c
          +++ new/usr/src/uts/common/contract/device.c
↓ open down ↓ 1915 lines elided ↑ open up ↑
1916 1916  
1917 1917          /*
1918 1918           * Release the lock only now so that the only point where we
1919 1919           * drop the lock is in wait_for_acks(). This is so that we don't
1920 1920           * miss cv_signal/cv_broadcast from contract holders
1921 1921           */
1922 1922          CT_DEBUG((CE_NOTE, "publish: dropping devi_ct_lock"));
1923 1923          mutex_exit(&(DEVI(dip)->devi_ct_lock));
1924 1924  
1925 1925  out:
1926      -        if (tnvl)
1927      -                nvlist_free(tnvl);
     1926 +        nvlist_free(tnvl);
1928 1927          if (path)
1929 1928                  kmem_free(path, MAXPATHLEN);
1930 1929  
1931 1930  
1932 1931          CT_DEBUG((CE_NOTE, "publish: result = %s", result_str(result)));
1933 1932          return (result);
1934 1933  }
1935 1934  
1936 1935  
1937 1936  /*
↓ open down ↓ 282 lines elided ↑ open up ↑
XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX