Opened 3 years ago

Closed 3 years ago

Last modified 3 years ago

#291 closed defect (fixed)

Crash when copying from font info box

Reported by: lewisr Owned by: gyoung
Priority: major Milestone:
Component: Lucide Core Version: 1.3.6
Keywords: Cc: stevenhl

Description

Copy via Ctrl+Ins one or multiple lines causes access violation in lucide1.dll.

Can reproduce with a variety of PDF documents. TRP file attached.

cc'ing Steve because, well, I know he enjoys these things.

Attachments (1)

0088_01.TRP (63.8 KB) - added by lewisr 3 years ago.

Download all attachments as: .zip

Change History (7)

Changed 3 years ago by lewisr

comment:1 Changed 3 years ago by gyoung

Ctr+C does it to. Not all pdfs show the trap. It could be related to the addition of substitute fonts in the dialog but I looked at all the code I added and don't see an obvious mistake.

comment:2 Changed 3 years ago by lewisr

I just found a PDF which does not crash (Ctrl+C or Ctrl+Ins) when copying. Might this have something to do with the length of the font name and what we are passing to Uclip?

FWIW, I am using:

uclip-0.3.0-1.oc00.pentium4

Last edited 3 years ago by lewisr (previous) (diff)

comment:3 Changed 3 years ago by gyoung

I figured out what was happening sometimes subname is NULL and in that case when it was added to the string it trapped. Now fixed.

comment:4 Changed 3 years ago by gyoung

  • Owner set to gyoung
  • Status changed from new to assigned

comment:5 Changed 3 years ago by gyoung

  • Resolution set to fixed
  • Status changed from assigned to closed

CS [545]

comment:6 Changed 3 years ago by lewisr

  • Summary changed from Crash when copying from font into box to Crash when copying from font info box

Copied font info from document which does not crash:

Helvetica | Type 1 | Not embedded | Helvetica
Symbol | Type 1 | Not embedded | Symbol
Helvetica-Bold | Type 1 | Not embedded | Helvetica Bold
Times-Bold | Type 1 | Not embedded | Times New Roman Negreta
Times-Roman | Type 1 | Not embedded | Times New Roman
Arial-BoldMT | Type 1 | Not embedded | Arial Negreta
TimesNewRomanPSMT | Type 1 | Not embedded | DejaVu Sans

Fonts in document which crash:

LiberationSerif-Bold | TrueType | Embedded subset
LiberationSerif      | TrueType | Embedded subset

This would prove your point (no subname).

Thanks.

Last edited 3 years ago by lewisr (previous) (diff)
Note: See TracTickets for help on using tickets.