* src/procmsg.c
authorChristoph Hohmann <reboot@gmx.ch>
Tue, 19 Nov 2002 12:56:36 +0000 (12:56 +0000)
committerChristoph Hohmann <reboot@gmx.ch>
Tue, 19 Nov 2002 12:56:36 +0000 (12:56 +0000)
        add parameter check for flag setting function
        (closes bug [ 640619 ] Claws169: crash
         - drafting a mail (IMAP))

we should still investigate why the function is called with
a null pointer

ChangeLog.claws
configure.in
src/procmsg.c

index d2253c7..bc12294 100644 (file)
@@ -1,3 +1,10 @@
+2002-11-19 [christoph] 0.8.5claws170
+
+       * src/procmsg.c
+               add parameter check for flag setting function
+               (closes bug [ 640619 ] Claws169: crash
+                - drafting a mail (IMAP))
+
 2002-11-18 [paul]      0.8.5claws169
 
        * src/compose.c
index 9e6ddfa..7a6202c 100644 (file)
@@ -11,7 +11,7 @@ MINOR_VERSION=8
 MICRO_VERSION=5
 INTERFACE_AGE=0
 BINARY_AGE=0
-EXTRA_VERSION=claws169
+EXTRA_VERSION=claws170
 VERSION=$MAJOR_VERSION.$MINOR_VERSION.$MICRO_VERSION$EXTRA_VERSION
 
 dnl set $target
index 02d014b..98e0edc 100644 (file)
@@ -1432,8 +1432,12 @@ msginfo->folder->folder->change_flags(msginfo->folder->folder, \
 
 void procmsg_msginfo_set_flags(MsgInfo *msginfo, MsgPermFlags perm_flags, MsgTmpFlags tmp_flags)
 {
-       FolderItem *item = msginfo->folder;
+       FolderItem *item;
 
+       g_return_if_fail(msginfo != NULL);
+       item = msginfo->folder;
+       g_return_if_fail(item != NULL);
+       
        debug_print("Setting flags for message %d in folder %s\n", msginfo->msgnum, item->path);
 
        /* if new flag is set */
@@ -1474,7 +1478,11 @@ void procmsg_msginfo_set_flags(MsgInfo *msginfo, MsgPermFlags perm_flags, MsgTmp
 
 void procmsg_msginfo_unset_flags(MsgInfo *msginfo, MsgPermFlags perm_flags, MsgTmpFlags tmp_flags)
 {
-       FolderItem *item = msginfo->folder;
+       FolderItem *item;
+
+       g_return_if_fail(msginfo != NULL);
+       item = msginfo->folder;
+       g_return_if_fail(item != NULL); 
        
        debug_print("Unsetting flags for message %d in folder %s\n", msginfo->msgnum, item->path);