From 9ef03874bf0ca35348c58b4b21cb06cc49026efd Mon Sep 17 00:00:00 2001 From: Selva Nair Date: Sun, 15 Oct 2017 15:05:15 -0400 Subject: [PATCH] Fix missing check for return value of malloc'd buffer - Use utf8to16 from common.c for utf8 to wide conversion and check its return value Signed-off-by: Selva Nair Acked-by: Gert Doering Message-Id: <1508094315-466-1-git-send-email-selva.nair@gmail.com> URL: https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg15641.html Signed-off-by: Gert Doering (cherry picked from commit f3d389a2d2b87aeb649bfdccd596f485346a32c7) --- src/openvpnserv/interactive.c | 18 ++++++++++-------- 1 file changed, 10 insertions(+), 8 deletions(-) diff --git a/src/openvpnserv/interactive.c b/src/openvpnserv/interactive.c index 0c91199a8..aab784e75 100644 --- a/src/openvpnserv/interactive.c +++ b/src/openvpnserv/interactive.c @@ -554,15 +554,17 @@ static DWORD InterfaceLuid(const char *iface_name, PNET_LUID luid) { NETIO_STATUS status; - LPWSTR wide_name; - int n; - - n = MultiByteToWideChar(CP_UTF8, 0, iface_name, -1, NULL, 0); - wide_name = malloc(n * sizeof(WCHAR)); - MultiByteToWideChar(CP_UTF8, 0, iface_name, -1, wide_name, n); - status = ConvertInterfaceAliasToLuid(wide_name, luid); - free(wide_name); + LPWSTR wide_name = utf8to16(iface_name); + if (wide_name) + { + status = ConvertInterfaceAliasToLuid(wide_name, luid); + free(wide_name); + } + else + { + status = ERROR_OUTOFMEMORY; + } return status; } -- 2.47.2