diff options
| author | Artem Senichev <a.senichev@yadro.com> | 2019-09-26 14:44:24 +0300 |
|---|---|---|
| committer | Daniel M Crowell <dcrowell@us.ibm.com> | 2019-10-03 12:32:37 -0500 |
| commit | c79620afdf079771a186d22743e19596ddada22d (patch) | |
| tree | d64acc0fdbb8b4991e6cc91548c5194c011faa58 /src | |
| parent | ad8653d6a2db95816c5f36d9bfabad959e8a4427 (diff) | |
| download | talos-hostboot-c79620afdf079771a186d22743e19596ddada22d.tar.gz talos-hostboot-c79620afdf079771a186d22743e19596ddada22d.zip | |
errl: Free memory allocated for parsers
Errl's factory creates new instance of User Defined Data parser
on every call, but doesn't free it after usage.
Resolves #184
Signed-off-by: Artem Senichev <a.senichev@yadro.com>
Change-Id: I8e8bd54e7aaa4ddb813cec3485404b6ba7a29ffa
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84518
Reviewed-by: Corey V Swenson <cswenson@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: Matt Derksen <mderkse1@us.ibm.com>
Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
Diffstat (limited to 'src')
| -rwxr-xr-x | src/usr/errl/plugins/errludparser.H | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/usr/errl/plugins/errludparser.H b/src/usr/errl/plugins/errludparser.H index ddaadade7..6957d4632 100755 --- a/src/usr/errl/plugins/errludparser.H +++ b/src/usr/errl/plugins/errludparser.H @@ -5,8 +5,9 @@ /* */ /* OpenPOWER HostBoot Project */ /* */ -/* Contributors Listed Below - COPYRIGHT 2012,2018 */ +/* Contributors Listed Below - COPYRIGHT 2012,2019 */ /* [+] International Business Machines Corp. */ +/* [+] YADRO */ /* */ /* */ /* Licensed under the Apache License, Version 2.0 (the "License"); */ @@ -84,6 +85,7 @@ static bool myDataParse(\ {\ l_rc = true;\ l_pParser->parse(i_ver, i_parser, i_buffer, i_buflen);\ + delete l_pParser;\ }\ return l_rc;\ }\ |
