From: Alin Serdean <[email protected]> The result of `RtlStringCbLengthW` is not currently checked and triggers a warning using the WDK 8.1 static analysis.
This patch treats the result of `RtlStringCbLengthW`. Signed-off-by: Alin Gabriel Serdean <[email protected]> --- v2: s/Fail/Failed. Suggested by: Shashank Ram [email protected] --- datapath-windows/ovsext/Vport.c | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/datapath-windows/ovsext/Vport.c b/datapath-windows/ovsext/Vport.c index 075f419..7cf2497 100644 --- a/datapath-windows/ovsext/Vport.c +++ b/datapath-windows/ovsext/Vport.c @@ -1144,8 +1144,13 @@ GetNICAlias(PNDIS_SWITCH_NIC_PARAMETERS nicParam, if (status == STATUS_SUCCESS) { RtlStringCbPrintfW(portFriendlyName->String, IF_MAX_STRING_SIZE, L"%s", interfaceName); - RtlStringCbLengthW(portFriendlyName->String, IF_MAX_STRING_SIZE, - &len); + status = RtlStringCbLengthW(portFriendlyName->String, + IF_MAX_STRING_SIZE, &len); + if (!NT_SUCCESS(status)) { + OVS_LOG_ERROR("Failed to get the length of the string," + "status: %x", status); + return status; + } portFriendlyName->Length = (USHORT)len; } else { OVS_LOG_ERROR("Fail to convert interface LUID to alias, status: %x", -- 2.10.2.windows.1 _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
