Dmitry Timoshkov wrote:
"Mikolaj Zalewski" <[EMAIL PROTECTED]> wrote:
+/* The validation is also done in explorer. However we must also
do it
+ * on the client size so that WM_COPYDATA doesn't crash the
application
+ */
+if (nid->cbSize != NOTIFYICONDATAW_V1_SIZE &&
+nid->cbSize != NOTIFYICONDATAW_V2_SIZE &&
+nid->cbSize != NOTIFYICONDATAW_V3_SIZE &&
+nid->cbSize != sizeof(NOTIFYICONDATAW))
+{
+WARN("Invalid size\n");
+return FALSE;
+}
Do you have a test case for this?
I was testing this but now I see my tests failed because the hWnd was
invalid. With a valid hWnd this works even for an invalid cbSize, so
this patch is wrong. I'll try to check tomorrow how that should be done
correctly.
Mikolaj Zalewski
