Closed Bug 384748 Opened 17 years ago Closed 17 years ago

xpidl can generate garbage .java filenames

Categories

(Core Graveyard :: Java to XPCOM Bridge, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: ajschult784, Assigned: ajschult784)

References

Details

(Keywords: fixed1.8.1.8, Whiteboard: [needs testcase])

Attachments

(1 file)

Attached patch patchSplinter Review
xpidl_java's interface_declaration can construct the output file from an uninitialized char* if IDL filename has no '/'.
Attachment #268655 - Flags: review?(jhpedemonte)
Attachment #268655 - Flags: review?(jhpedemonte) → review+
FIXED
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Comment on attachment 268655 [details] [diff] [review]
patch

Asking for 1.8.1.7 approval.  This is a low risk change that fixes a bug when using XPIDL to generate Java interface files.
Attachment #268655 - Flags: approval1.8.1.7?
Sorry about the duplicate report.

As a minor comment, this code:

     strcat(outname, interface_name);
     strcat(outname, ".java");

may possibly overflow the outname buffer. My patch in bug 390705 tries to be more careful there.
Comment on attachment 268655 [details] [diff] [review]
patch

approved for 1.8.1.7, a=dveditz for release-drivers
Attachment #268655 - Flags: approval1.8.1.7? → approval1.8.1.7+
Keywords: fixed1.8.1.7
Whiteboard: [needs testcase]
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: