* 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 d2253c7c495cbc840d54552497c03b6d3d5c90b0..bc12294292ad308d7e48d4f8881a9236858c1034 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 9e6ddfa76da4027d70a493b4f5d9e6f3bb02b784..7a6202c439f05dfc95bd193b852089dd409e42ca 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 02d014b4345c9dbf9e4a90da1ff828b69ac6a815..98e0edce80657f4568ebb004b229bfcebe368f63 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);