From: Mehdi Abaakouk <sileht@sileht.net>
Date: Thu, 13 Dec 2018 15:01:36 +0000 (+0100)
Subject: imap: use imap_mailbox_status() in imap_mbox_check_stats()
X-Git-Tag: 2019-10-25~437
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=156d9e398b16b548ffbbcb5713d1d1e23c9db376;p=neomutt

imap: use imap_mailbox_status() in imap_mbox_check_stats()

This change calls imap_mailbox_status() instead of imap_status()
in imap_mbox_check_stats() as we known the Mailbox.

This also add more sanity check in imap_mailbox_status().
---

diff --git a/imap/imap.c b/imap/imap.c
index 2de5a439d..8efc3c915 100644
--- a/imap/imap.c
+++ b/imap/imap.c
@@ -1309,10 +1309,7 @@ static int imap_status(struct ImapAccountData *adata, struct ImapMboxData *mdata
  */
 int imap_mbox_check_stats(struct Mailbox *m, int flags)
 {
-  struct ImapAccountData *adata = imap_adata_get(m);
-  struct ImapMboxData *mdata = imap_mdata_get(m);
-
-  int rc = imap_status(adata, mdata, true);
+  int rc = imap_mailbox_status(m, true);
   if (rc > 0)
     rc = 0;
   return rc;
@@ -1351,9 +1348,11 @@ int imap_path_status(const char *path, bool queue)
  */
 int imap_mailbox_status(struct Mailbox *m, bool queue)
 {
-  if (!m)
+  struct ImapAccountData *adata = imap_adata_get(m);
+  struct ImapMboxData *mdata = imap_mdata_get(m);
+  if (!adata || !mdata)
     return -1;
-  return imap_status(m->account->adata, m->mdata, queue);
+  return imap_status(adata, mdata, queue);
 }
 
 /**