Marcus Meissner : ntdll: Potential buffer overwrite (Coverity).

Alexandre Julliard julliard at wine.codeweavers.com
Mon Apr 20 08:24:42 CDT 2015


Module: wine
Branch: master
Commit: 33d4125a8f471ab52455051479609cbb6da9ca0a
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=33d4125a8f471ab52455051479609cbb6da9ca0a

Author: Marcus Meissner <marcus at jet.franken.de>
Date:   Sun Apr 19 11:59:22 2015 +0200

ntdll: Potential buffer overwrite (Coverity).

---

 dlls/ntdll/cdrom.c | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/dlls/ntdll/cdrom.c b/dlls/ntdll/cdrom.c
index 34910e7..25a3fde 100644
--- a/dlls/ntdll/cdrom.c
+++ b/dlls/ntdll/cdrom.c
@@ -3090,8 +3090,11 @@ NTSTATUS CDROM_DeviceIoControl(HANDLE hDevice,
         sz = sizeof(DVD_REGION);
         if (lpInBuffer != NULL || nInBufferSize != 0) status = STATUS_INVALID_PARAMETER;
         else if (nOutBufferSize < sz) status = STATUS_BUFFER_TOO_SMALL;
-        TRACE("doing DVD_Get_REGION\n");
-        status = DVD_GetRegion(fd, lpOutBuffer);
+        else
+        {
+            TRACE("doing DVD_Get_REGION\n");
+            status = DVD_GetRegion(fd, lpOutBuffer);
+        }
         break;
     case IOCTL_DVD_READ_STRUCTURE:
         sz = sizeof(DVD_LAYER_DESCRIPTOR);




More information about the wine-cvs mailing list