Skip to content

Commit 856f1ad

Browse files
aalugorewghoffa
authored andcommitted
Fix memory leak by committing error log in eepromPresence()
Change-Id: I487276ddd7a15f61f48a69a239d61beacfe4886f ForwardPort: yes Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/23852 Tested-by: Jenkins Server Tested-by: Jenkins OP Build CI Tested-by: Jenkins OP HW Tested-by: FSP CI Jenkins Reviewed-by: Matthew A. Ploetz <maploetz@us.ibm.com> Reviewed-by: Prachi Gupta <pragupta@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com> Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com>
1 parent e189edd commit 856f1ad

File tree

1 file changed

+6
-0
lines changed

1 file changed

+6
-0
lines changed

src/usr/i2c/eepromdd.C

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -360,6 +360,12 @@ bool eepromPresence ( TARGETING::Target * i_target )
360360

361361
} while( 0 );
362362

363+
// If there was an error commit the error log
364+
if( err )
365+
{
366+
errlCommit( err, I2C_COMP_ID );
367+
}
368+
363369
TRACDCOMP(g_trac_eeprom, EXIT_MRK"eepromPresence()");
364370
return l_present;
365371
}

0 commit comments

Comments
 (0)