Bug #24274--> when creating a new channel, if we don't have a mock channel,

return an error instead of NS_OK.
r=bienvenu,travis
a=chofmann
This commit is contained in:
mscott%netscape.com 2000-01-20 03:55:25 +00:00
parent 2fc75424f3
commit 3f81da7c45

View File

@ -2372,7 +2372,7 @@ NS_IMETHODIMP nsImapService::NewChannel(const char *verb,
// XXX this mock channel stuff is wrong -- the channel really should be owning the URL
// and the originalURL, not the other way around
rv = imapUrl->GetMockChannel(getter_AddRefs(mockChannel));
if (NS_FAILED(rv) || !mockChannel) return rv;
if (NS_FAILED(rv) || !mockChannel) return NS_ERROR_FAILURE;
// XXX this seems wrong to do to this shared mock channel too,
// but oh well...