Closed Bug 102461 Opened 23 years ago Closed 23 years ago

Workaround for bug 93771 causes hang in Xprint module

Categories

(Core Graveyard :: Printing: Xprint, defect, P1)

All
Linux
defect

Tracking

(Not tracked)

VERIFIED FIXED
mozilla0.9.6

People

(Reporter: roland.mainz, Assigned: roland.mainz)

References

()

Details

Attachments

(1 file)

An attempt to print the example URL via Xprint module causes it to hang in an
endless loop.
The issue is burried in the hack-workaround for bug 93771, patch follows...
QA <--> Owner switch ...
Assignee: katakai → Roland.Mainz
QA Contact: Roland.Mainz → katakai
Accepting bug ...
Status: NEW → ASSIGNED
Priority: -- → P1
Target Milestone: --- → mozilla0.9.5
Attachment #51481 - Attachment description: Patch for bug 2001-09-28-08-trunk → Patch for 2001-09-28-08-trunk
This issue was caused by the fact that I restored the pointer to the string
object instead of restoring the string object's _value_.

Filed patch, requesting r=/sr= ...

bstell, can you give me an r=, please ?
Keywords: patch, review
Duh. And I r='ed the original code too, I think. r=jag
Attachment #51481 - Flags: review+
> Duh. And I r='ed the original code too, I think. 

Don't wrorry... I wrote this xx@@!!...
... and jesup, dcran and blizzard did not saw it, too... =:-)

> r=jag

Thanks!

----

Requesting sr= ...
Comment on attachment 51481 [details] [diff] [review]
Patch for 2001-09-28-08-trunk

sr=jst
Attachment #51481 - Flags: superreview+
CC:'ing timeless@bemail.org for checkin, please ...
Comment on attachment 51481 [details] [diff] [review]
Patch for 2001-09-28-08-trunk

a=asa (on behalf of drivers) for checkin to 0.9.5.
Attachment #51481 - Flags: approval+
Checked into the trunk and the branch
0.9.5 is out the door. bumping TM up by one.
Target Milestone: mozilla0.9.5 → mozilla0.9.6
Patch has been checked-in, marking bug as FIXED...
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
I can not see the hang on 2002011614 build.
Marked this as VERIFIED.
Status: RESOLVED → VERIFIED
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: