diff options
author | Hao Wu <hao.a.wu@intel.com> | 2016-09-27 13:40:58 +0800 |
---|---|---|
committer | Hao Wu <hao.a.wu@intel.com> | 2016-11-08 16:37:17 +0800 |
commit | fd5425230ed22872126b52f22a7294e352ca3349 (patch) | |
tree | 2a4820e880131f8d83c68deddc064cbfc896e1b1 /BaseTools/Source/C/VfrCompile/VfrFormPkg.cpp | |
parent | a6ac965bca117ef33b38a96c36643b36757b7698 (diff) | |
download | edk2-platforms-fd5425230ed22872126b52f22a7294e352ca3349.tar.xz |
BaseTools/VfrCompile: Avoid freeing memory with mismatched functions
Memory allocated by operator new[] should be freed using delete[] to avoid
possible memory leak.
Cc: Liming Gao <liming.gao@intel.com>
Cc: Yonghong Zhu <yonghong.zhu@intel.com>
Cc: Eric Dong <eric.dong@intel.com>
Cc: Dandan Bi <dandan.bi@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Hao Wu <hao.a.wu@intel.com>
Reviewed-by: Liming Gao <liming.gao@intel.com>
Diffstat (limited to 'BaseTools/Source/C/VfrCompile/VfrFormPkg.cpp')
-rw-r--r-- | BaseTools/Source/C/VfrCompile/VfrFormPkg.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/BaseTools/Source/C/VfrCompile/VfrFormPkg.cpp b/BaseTools/Source/C/VfrCompile/VfrFormPkg.cpp index 124b8e8e72..9c76b29418 100644 --- a/BaseTools/Source/C/VfrCompile/VfrFormPkg.cpp +++ b/BaseTools/Source/C/VfrCompile/VfrFormPkg.cpp @@ -56,13 +56,13 @@ SPendingAssign::~SPendingAssign ( )
{
if (mKey != NULL) {
- delete mKey;
+ delete[] mKey;
}
mAddr = NULL;
mLen = 0;
mLineNo = 0;
if (mMsg != NULL) {
- delete mMsg;
+ delete[] mMsg;
}
mNext = NULL;
}
@@ -898,7 +898,7 @@ CFormPkg::DeclarePendingQuestion ( strcpy (NewStr, SName);
strcat (NewStr, VarStr + strlen (FName));
ReturnCode = lCVfrVarDataTypeDB.GetDataFieldInfo (NewStr, Info.mInfo.mVarOffset, Info.mVarType, Info.mVarTotalSize);
- delete NewStr;
+ delete[] NewStr;
}
} else {
ReturnCode = VFR_RETURN_UNSUPPORTED;
|