Print this page
patch tsoome-feedback

*** 197,207 **** break; } if (nvlist_alloc(&rsrc, NV_UNIQUE_NAME|NV_UNIQUE_NAME_TYPE, 0) != 0) { for (i = 0; i < n; i++) { - if (hcl[i] != NULL) nvlist_free(hcl[i]); } fmd_hdl_free(hdl, hcl, sizeof (nvlist_t *) * n); } --- 197,206 ----
*** 209,219 **** nvlist_add_string(rsrc, FM_FMRI_SCHEME, FM_FMRI_SCHEME_HC) != 0 || nvlist_add_string(rsrc, FM_FMRI_HC_ROOT, "") != 0 || nvlist_add_uint32(rsrc, FM_FMRI_HC_LIST_SZ, n) != 0 || nvlist_add_nvlist_array(rsrc, FM_FMRI_HC_LIST, hcl, n) != 0) { for (i = 0; i < n; i++) { - if (hcl[i] != NULL) nvlist_free(hcl[i]); } fmd_hdl_free(hdl, hcl, sizeof (nvlist_t *) * n); nvlist_free(rsrc); } --- 208,217 ----
*** 227,237 **** fmd_case_solve(hdl, cp); nvlist_free(fru); } for (i = 0; i < n; i++) { - if (hcl[i] != NULL) nvlist_free(hcl[i]); } fmd_hdl_free(hdl, hcl, sizeof (nvlist_t *) * n); nvlist_free(rsrc); --- 225,234 ----
*** 397,407 **** rsrc = gmem_find_dimm_rsc(hdl, dimm->dimm_serial); flt = fmd_nvl_create_fault(hdl, GMEM_FAULT_DIMM_PAGES, GMEM_FLTMAXCONF, NULL, gmem_dimm_fru(dimm), rsrc); fmd_case_add_suspect(hdl, cp, flt); fmd_case_solve(hdl, cp); - if (rsrc != NULL) nvlist_free(rsrc); } } /* --- 394,403 ----
*** 433,443 **** NULL, gmem_dimm_fru(dimm), rsrc); dimm->dimm_flags |= GMEM_F_FAULTING; gmem_dimm_dirty(hdl, dimm); fmd_case_add_suspect(hdl, cp, flt); fmd_case_solve(hdl, cp); - if (rsrc != NULL) nvlist_free(rsrc); return; } } } --- 429,438 ----
*** 703,713 **** } dimm->dimm_flags |= GMEM_F_FAULTING; gmem_dimm_dirty(hdl, dimm); fmd_case_add_suspect(hdl, dimm->dimm_case.cc_cp, flt); fmd_case_solve(hdl, dimm->dimm_case.cc_cp); - if (rsc != NULL) nvlist_free(rsc); return; } upos_pairs = i; assert(upos_pairs < 16); --- 698,707 ----