Fwd: New Defects reported by Coverity Scan for Thunderbird
It has been a while since the last time Coverity found problems in
TB-specific code (well, I am in the middle of PC hardware issue so I may
have missed other relevant coverity e-mail in the last 15 days or so.).
But all the problems reported in the following e-mail seem to be related
to TB and worth investigating.
I am sending this to development list just in case pertinent people are
not receiving this e-mail.
-------- Forwarded Message --------
Subject: New Defects reported by Coverity Scan for Thunderbird
Date: Wed, 24 Oct 2018 02:48:47 +0000 (UTC)
From: [hidden email] To: [hidden email]
Please find the latest report on new defect(s) introduced to Thunderbird found with Coverity Scan.
6 new defect(s) introduced to Thunderbird found with Coverity Scan.
3655 defect(s), reported by Coverity Scan earlier, were marked fixed in the recent build analyzed by Coverity Scan.
New defect(s) Reported-by: Coverity Scan
Showing 6 of 6 defect(s)
*** CID 1260145: Null pointer dereferences (FORWARD_NULL)
/comm/mailnews/imap/src/nsImapProtocol.cpp: 7032 in nsImapProtocol::RenameHierarchyByHand(const char *, const char *)()
7027 // calculate the new name and do the rename
7028 nsCString newChildName(newParentMailboxName);
7029 newChildName += (currentName + PL_strlen(oldParentMailboxName));
7030 // Pass in 'nonHierarchicalRename' to determine if we should really
7031 // reanme, or just move subscriptions.
>>> CID 1260145: Null pointer dereferences (FORWARD_NULL)
>>> Passing null pointer "currentName" to "RenameMailboxRespectingSubscriptions", which dereferences it.
7032 renameSucceeded =