Re: [PATCH] Re: help fixing up SetConsoleCtrlHandler()

[Peter Riocreux]

"Dimitrie O. Paun"  writes:
> On April 15, 2004 7:58 am, Peter Riocreux wrote:
> > Here is my first cut.
> >
> > Remember that this is my first patch, so go easy on me with he
> > flamethrowers. I know the indenting isn't quite right.
>
> The formatting doesn't seem too bad, but the format is wrong.
> Please use uniffied diff (-u):
>     http://www.winehq.org/site/sending_patches

D'oh! I did the first time I generated the patch, but forgot this
time. I *had* read that page, but ..... old age don' you know. :-^

I will include at the end the new patch.
> > I think that, despite what Juan Lang advised, I need to detect version
> > numbers as the FIXME can be triggered by perfecly legal use in
> > XP/NT/?? but illegal code in ME/98/95.
>
> Then allow it. Are there apps that depend on the 9x behaviour
> of not allowing it?

I don't know, I just had thought that Wine was attempting to be
bug-for-bug compatible, and assumed that part of that was honouring
all the restrictions on usage of functions and how these varied
between versions.

I am not seeing this FIXME coming out, so I am not concerned with
leaving it in, but have no strong opinion.

Peter

Index: dlls/kernel/console.c
===================================================================
RCS file: /home/wine/wine/dlls/kernel/console.c,v
retrieving revision 1.30
diff -u -r1.30 console.c
--- dlls/kernel/console.c	13 Apr 2004 21:16:26 -0000	1.30
+++ dlls/kernel/console.c	15 Apr 2004 12:08:08 -0000
@@ -1448,17 +1448,20 @@
 {
     BOOL        ret = TRUE;

-    FIXME("(%p,%i) - no error checking or testing yet\n", func, add);
-
     if (!func)
     {
+        FIXME("(%p,%i) - this is not allowed in WinME, Win98 or Win95\n", func, add);
    CONSOLE_IgnoreCtrlC = add;
     }
     else if (add)
     {
         struct ConsoleHandler*  ch = HeapAlloc(GetProcessHeap(), 0, sizeof(struct ConsoleHandler));

-        if (!ch) return FALSE;
+        if (!ch)
+	{
+	    SetLastError(ERROR_NOT_ENOUGH_MEMORY);
+	    return FALSE;
+	}
         ch->handler = func;
         RtlEnterCriticalSection(&CONSOLE_CritSect);
         ch->next = CONSOLE_Handlers;
@@ -1467,34 +1470,37 @@
     }
     else
     {
-        struct ConsoleHandler**  ch;
         RtlEnterCriticalSection(&CONSOLE_CritSect);
-        for (ch = &CONSOLE_Handlers; *ch; *ch = (*ch)->next)
-        {
-            if ((*ch)->handler == func) break;
-        }
-        if (*ch)
-        {
-            struct ConsoleHandler*   rch = *ch;
+	if (func == CONSOLE_DefaultHandler)
+	{
+	    ERR("Attempt to remove default CtrlHandler %p\n", func);
+	    SetLastError(ERROR_FUNCTION_FAILED);
+	    ret = FALSE;
+	}
+	else
+	{
+	    struct ConsoleHandler* ch, * prev ;
+	    prev = NULL;
+	    for (ch = CONSOLE_Handlers; ch; prev = ch, ch = ch->next)
+	    {
+	        if (ch->handler == func)
+		{
+		    if (ch == CONSOLE_Handlers)
+		        CONSOLE_Handlers = ch->next;
+		    else
+		        prev->next = ch->next;
+		    HeapFree(GetProcessHeap(), 0, ch);
+		    break;
+		}
+	    }

-            /* sanity check */
-            if (rch == &CONSOLE_DefaultConsoleHandler)
-            {
-                ERR("Who's trying to remove default handler???\n");
-                ret = FALSE;
-            }
-            else
-            {
-                rch = *ch;
-                *ch = (*ch)->next;
-                HeapFree(GetProcessHeap(), 0, rch);
-            }
-        }
-        else
-        {
-            WARN("Attempt to remove non-installed CtrlHandler %p\n", func);
-            ret = FALSE;
-        }
+	    if (! ch)
+	    {
+	        WARN("Attempt to remove non-installed CtrlHandler %p\n", func);
+		SetLastError(ERROR_INVALID_ADDRESS);
+		ret = FALSE;
+	    }
+	}
         RtlLeaveCriticalSection(&CONSOLE_CritSect);
     }
     return ret;