/** @file\r
DevicePathFromText protocol as defined in the UEFI 2.0 specification.\r
\r
-Copyright (c) 2017, Intel Corporation. All rights reserved.<BR>\r
+Copyright (c) 2017 - 2018, Intel Corporation. All rights reserved.<BR>\r
This program and the accompanying materials\r
are licensed and made available under the terms and conditions of the BSD License\r
which accompanies this distribution. The full text of the license may be found at\r
\r
ISCSIDevPath->LoginOption = (UINT16) Options;\r
\r
- if (StrCmp (ProtocolStr, L"TCP") == 0) {\r
+ if (IS_NULL (*ProtocolStr) || (StrCmp (ProtocolStr, L"TCP") == 0)) {\r
ISCSIDevPath->NetworkProtocol = 0;\r
} else {\r
//\r
//\r
FromText = DevPathFromTextFilePath;\r
DeviceNode = FromText (DeviceNodeStr);\r
+ //\r
+ // According to above logic, if 'FromText' is NULL in the 'if' statement,\r
+ // then 'ParamStr' must be NULL as well. No memory allocation has been made\r
+ // in this case.\r
+ //\r
+ // The below check is for addressing a false positive potential memory leak\r
+ // issue raised from static analysis.\r
+ //\r
+ if (ParamStr != NULL) {\r
+ free (ParamStr);\r
+ }\r
} else {\r
DeviceNode = FromText (ParamStr);\r
free (ParamStr);\r