Fix potential null pointer deferenced (#363)

* Fix potential null pointer deferenced clist_content is a MACRO and return NULL: #define clist_content(iter) (iter ? (iter)->data : NULL) * FIx invalid test against NULL.
dinhvh · Apr 30, 2020 · 68bde8b · 68bde8b
1 parent ae072d8
commit 68bde8b
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/src/driver/implementation/imap/imapdriver_tools.c b/src/driver/implementation/imap/imapdriver_tools.c
@@ -1716,7 +1716,7 @@ static int imap_mailbox_list_to_group(clist * imap_mb_list, clistiter ** iter,
   imap_mailbox_listiter = * iter;
 
   imap_addr = clist_content(imap_mailbox_listiter);
-  if (imap_addr->ad_mailbox_name == NULL) {
+  if (imap_addr == NULL || imap_addr->ad_mailbox_name == NULL) {
     res = MAIL_ERROR_INVAL;
     goto err;
   }