Closed Bug 104938 Opened 23 years ago Closed 23 years ago

CRASH: Routing Recipient causes MS Word to crash.

Categories

(MailNews Core :: Simple MAPI, defect)

x86
Windows 2000
defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: trix, Assigned: kkhandrika)

Details

(Keywords: crash, Whiteboard: [PDT+])

Attachments

(2 files)

With Netscape Mail as the default, in MS Word if you select "Routing Recipient"
instead of "Mail Recipient(as Attachment)...", an application error will occur.

DIRECTIONS:
1. Set Netscape Mail to be your default email client.
2. Open MS Word(don't need to enter anything into the document)
3. Select File|Send To|Routing Recipient

ACTUAL RESULTS:
An application error occurs causing MS Word to crash.

EXPECTED RESULTS:
With Outlook Express as the default, the Routing Slip dialog is displayed.

With Communicator 4.x as the default, a "General mail failure.  Close Microsoft
Word, restart the mail system, and try again." which is an error dialog from MS
Word.

At least with 4.x, we exit gracefully.
Just an additional note.  This is not isolated to just MS Word, but Excel
displays the same problem.  On Powerpoint though, it exits gracefully with an
error dialog "Powerpoint found an error and it can't correct...", exiting the
app does allow you to save the doc and exit.
The problem is with returning SUCCESS_SUCCESS instead of MAPI_E_FAILURE for
unsupported functionalities in this release.   There is no need to return
MAPI_E_FAILURE from the functions those do not have OUT parameter, still I feel
it is better to let the app. know about it.

Rajiv  - Can you please review
mscott - Can you please super review the same

Thanks
Forgot to CC Rajiv (Reviewer)
good, this patch is easier to review.

r=rdayal.

Since this could help in removing probable crashes when calls are made to
non-implemented Simple MAPI functions by the mapi apps, am nominating this for 
094 branch. A simple no brainer kinda change/fix.
What does the user see with this patch for these unsupported items?
Keywords: crash, nsbranch
MAPI is a set of APIs which returns error codes, what the user sees is completly
based on what the MAPI apps displays to the user when they get these return codes. 
Comment on attachment 53678 [details] [diff] [review]
New patch with more context

sr=mscott. Not really much for me to review here. I'm assuming with these changes our 4 basic mapi clients still work....
Attachment #53678 - Flags: superreview+
pls get this in tonight = PDT+
Whiteboard: [PDT+]
Attachment #53678 - Flags: review+
This wasn't checked in last night?  
Will be checked in today.
hmmmm ... we didn't check this in last night?
Checked into 0.9.4 branch
Trix:

Please verify this bug on tomorrow's build ASAP. Many thanks.
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
verified on 2001-10-17-05-0.9.4

General Mail Failure warning dialog is now displayed instead of crashing which
is the same behavior as Comm. 4.x.
Status: RESOLVED → VERIFIED
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: