When freeing XMS memory block, return an error if block has not been

previously allocated.

When freeing XMS memory block, return an error if block has not been
6138ba08 · Jukka Heinonen · Alexandre Julliard · 30678c33 · 6138ba08
Commit 6138ba08 authored Apr 29, 2002 by Jukka Heinonen Committed by Alexandre Julliard Apr 29, 2002
Hide whitespace changes
Inline Side-by-side

Showing with 5 additions and 1 deletion

xms.c dlls/winedos/xms.c +5 -1

No files found.
--- a/dlls/winedos/xms.c
+++ b/dlls/winedos/xms.c
@@ -84,7 +84,11 @@ void WINAPI XMS_Handler( CONTEXT86 *context )
 	break;
    case 0x0a:   /* Free Extended Memory Block */
 	TRACE("free extended memory block %04x\n",DX_reg(context));
-	GlobalFree16(DX_reg(context));
+       if(!DX_reg(context) || GlobalFree16(DX_reg(context))) {
+         AX_reg(context) = 0;    /* failure */
+         BL_reg(context) = 0xa2; /* invalid handle */
+       } else
+         AX_reg(context) = 1;    /* success */
 	break;
    case 0x0b:   /* Move Extended Memory Block */
    {