commit ebf1243ec15a42384be4d4b9af7e16d974ebb21b
author Alexandre Julliard <julliard@winehq.org> Wed Dec 10 01:34:51 2003 +0000
committer Alexandre Julliard <julliard@winehq.org> Wed Dec 10 01:34:51 2003 +0000
tree 77af54c40b3522015bbc33dc23028fb41b3d03fc
parent d5b4232a3f2385b80156d56314e0fbe75c99423c

Added support for inter-process GetWindowWord.
Do better range checking on the window extra bytes.

server/window.c

diff --git a/server/window.c b/server/window.c
index ff16793..6373068 100644
--- a/server/window.c
+++ b/server/window.c
@@ -586,20 +586,20 @@
         set_error( STATUS_ACCESS_DENIED );
         return;
     }
-    if (req->extra_offset < -1 || req->extra_offset >= win->nb_extra_bytes)
+    if (req->extra_size > sizeof(req->extra_value) ||
+        req->extra_offset < -1 ||
+        req->extra_offset > win->nb_extra_bytes - (int)req->extra_size)
     {
-        set_error( STATUS_INVALID_PARAMETER );
+        set_win32_error( ERROR_INVALID_INDEX );
         return;
     }
     if (req->extra_offset != -1)
     {
-        memcpy( &reply->old_extra_value, win->extra_bytes + req->extra_offset,
-                min( sizeof(reply->old_extra_value),
-                     (size_t)(win->nb_extra_bytes - req->extra_offset) ));
+        memcpy( &reply->old_extra_value, win->extra_bytes + req->extra_offset, req->extra_size );
     }
-    else if (req->flags & (SET_WIN_EXTRAWORD|SET_WIN_EXTRALONG))
+    else if (req->flags & SET_WIN_EXTRA)
     {
-        set_error( STATUS_INVALID_PARAMETER );
+        set_win32_error( ERROR_INVALID_INDEX );
         return;
     }
     reply->old_style     = win->style;
@@ -612,12 +612,8 @@
     if (req->flags & SET_WIN_ID) win->id = req->id;
     if (req->flags & SET_WIN_INSTANCE) win->instance = req->instance;
     if (req->flags & SET_WIN_USERDATA) win->user_data = req->user_data;
-    if (req->flags & (SET_WIN_EXTRAWORD|SET_WIN_EXTRALONG))
-    {
-        const int len = (req->flags & SET_WIN_EXTRALONG) ? sizeof(int) : sizeof(short);
-        memcpy( win->extra_bytes + req->extra_offset, &req->extra_value,
-                min( len, win->nb_extra_bytes - req->extra_offset ));
-    }
+    if (req->flags & SET_WIN_EXTRA) memcpy( win->extra_bytes + req->extra_offset,
+                                            &req->extra_value, req->extra_size );
 }