2001-04-10  Matt Hargett  <[EMAIL PROTECTED]>

* ksound.c: Added NULL pointer checks.
* Initialized a couple of pointers to NULL on declaration.
* (kDoSound): Added default to switch.

--
http://www.clock.org/~matt




-- Attached file included as plaintext by Listar --
-- File: ksound.c.diff

--- ksound.c.old        Tue Apr 10 17:12:55 2001
+++ ksound.c    Tue Apr 10 17:20:54 2001
@@ -44,7 +44,13 @@
 void
 process_sound_events(state_t *s) /* Get all sound events, apply their changes to the 
heap */
 {
-       sound_event_t *event;
+       sound_event_t *event    = NULL;
+
+       if (NULL == s)
+       {
+               sciprintf("ksound.c: process_sound_events(): NULL passed for parameter 
+s\n");
+               return;
+       }
 
        if (s->sfx_driver == NULL)
                return;
@@ -117,6 +123,12 @@ process_sound_events(state_t *s) /* Get 
 void
 kDoSound(state_t *s, int funct_nr, int argc, heap_ptr argp)
 {
+       if (NULL == s)
+       {
+               sciprintf("ksound.c: process_sound_events(): NULL passed for parameter 
+s\n");
+               return;
+       }
+
        word command = UPARAM(0);
        heap_ptr obj = UPARAM_OR_ALT(1, 0);
 
@@ -142,6 +154,7 @@ kDoSound(state_t *s, int funct_nr, int a
                case 10: sciprintf("[Fade(?)]"); break;
                case 11: sciprintf("[ChkDriver]"); break;
                case 12: sciprintf("[StopAll]"); break;
+               default: sciprintf("[Unknown]"); break;
                }
 
                sciprintf("(");


Reply via email to