Changeset 304

Timestamp:

Nov 13, 2010, 10:51:12 PM (14 years ago)

Author:

katerina

Message:

Fix for ticket #228 (null poiter dereference in handler for SetMailAlias).

Location:

trunk

Files:

• docs/Changelog (modified) (1 diff)
• src/sh_nmail.c (modified) (1 diff)

trunk/docs/Changelog

@@ -1 +1 @@
 2.8.1:
+        * Fix null pointer dereference in config parse handler for SetMailAlias
+          (reported by Sergey)
         * Fix incorrect memset() in sh_kern.c (passed struct by value...),
           reported by Roman and Stefan

trunk/src/sh_nmail.c

@@ -234 +234 @@
               SH_FREE(array);
 
-              if (newalias && newalias->recipient_list == NULL)
+              if (newalias == NULL || newalias->recipient_list == NULL)
                 {
-                  SH_FREE(newalias);
+                  if (newalias)
+                    SH_FREE(newalias);
                   goto err;
                 }