summaryrefslogtreecommitdiffstats
path: root/src/usr/targeting
diff options
context:
space:
mode:
authorChristian Geddes <crgeddes@us.ibm.com>2017-09-08 10:22:52 -0500
committerDaniel M. Crowell <dcrowell@us.ibm.com>2017-09-25 11:54:57 -0400
commit1b3b999258db544d6af1dbfb7ee3437efd17581b (patch)
tree1cb96dd1838af832ef630398c93cfedeaafda3ef /src/usr/targeting
parentac058e9b3c6d1d5d57089d42cbb75b76f36d24a2 (diff)
downloadtalos-hostboot-1b3b999258db544d6af1dbfb7ee3437efd17581b.tar.gz
talos-hostboot-1b3b999258db544d6af1dbfb7ee3437efd17581b.zip
Set variables to nullptr after they are deleted
While updating some LPC logic a developer noticed a bug where an errlog handle wasnt getting set to nullptr after it was deleted this caused a confusing bug that took awhile to track. I noticed that this bug was all over our code so this commit fixes a lot of cases where variables get deleted but not set to nullptr Change-Id: I103b5c71b93686c3c89b04d1d565a24d4de74e6f Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/45890 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Reviewed-by: Matt Derksen <mderkse1@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
Diffstat (limited to 'src/usr/targeting')
-rw-r--r--src/usr/targeting/runtime/attrPlatOverride_rt.C1
1 files changed, 1 insertions, 0 deletions
diff --git a/src/usr/targeting/runtime/attrPlatOverride_rt.C b/src/usr/targeting/runtime/attrPlatOverride_rt.C
index fcd6aabc1..b89c93bb4 100644
--- a/src/usr/targeting/runtime/attrPlatOverride_rt.C
+++ b/src/usr/targeting/runtime/attrPlatOverride_rt.C
@@ -167,6 +167,7 @@ void applyTempOverrides()
// if the new RHB is in use, this will always fail
// so just delete the error
delete l_err;
+ l_err = nullptr;
}
else
{
OpenPOWER on IntegriCloud