diff options
| author | Toomas Soome <tsoome@me.com> | 2019-12-30 15:05:56 +0200 |
|---|---|---|
| committer | Toomas Soome <tsoome@me.com> | 2020-03-16 08:57:02 +0200 |
| commit | 584b574a3b16c6772c8204ec1d1c957c56f22a87 (patch) | |
| tree | c9bfd155a2c2edf14608a4ee0d1b328680cde6d8 /usr/src/uts/intel/os/fmsmb.c | |
| parent | 34a4e6b53d2e6f2605fd77cda5b161201d7d0f20 (diff) | |
| download | illumos-gate-584b574a3b16c6772c8204ec1d1c957c56f22a87.tar.gz | |
12174 i86pc: variable may be used uninitialized
Reviewed by: John Levon <john.levon@joyent.com>
Reviewed by: Andrew Stormont <astormont@racktopsystems.com>
Approved by: Dan McDonald <danmcd@joyent.com>
Diffstat (limited to 'usr/src/uts/intel/os/fmsmb.c')
| -rw-r--r-- | usr/src/uts/intel/os/fmsmb.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/usr/src/uts/intel/os/fmsmb.c b/usr/src/uts/intel/os/fmsmb.c index 3a4785475b..51ca9dd693 100644 --- a/usr/src/uts/intel/os/fmsmb.c +++ b/usr/src/uts/intel/os/fmsmb.c @@ -1073,7 +1073,8 @@ bad: } int -fm_smb_mc_chipinst(uint_t bdf, uint_t *chip_inst) { +fm_smb_mc_chipinst(uint_t bdf, uint_t *chip_inst) +{ int i, j; smbios_hdl_t *shp; @@ -1101,10 +1102,12 @@ fm_smb_mc_chipinst(uint_t bdf, uint_t *chip_inst) { mastypes->type = SUN_OEM_EXT_MEMARRAY; smb_strcnt(shp, mastypes); + pstypes = NULL; + p_strcnt = 0; for (i = 0; i < mastypes->count; i++) { ext_id = mastypes->ids[i]->id; (void) smbios_info_extmemarray(shp, ext_id, &em); - if (em.smbmae_bdf == bdf) { + if (em.smbmae_bdf == bdf) { p_strcnt = smb_cnttypes(shp, SMB_TYPE_PROCESSOR); if (p_strcnt == 0) { smb_free_strcnt(mastypes, ma_strcnt); |