(bad render) incorrect float parsing in data attribute in tag <path>

Bug #259485 reported by Actam
This bug report is a duplicate of:  Bug #370061: SVG not correctly rendered. Edit Remove
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
librsvg
Fix Released
Unknown
librsvg (Ubuntu)
Incomplete
Low
Ubuntu Desktop Bugs

Bug Description

Binary package hint: librsvg2-2

Package: librsvg2-2 version 2.22.2-2

The is a parsing error in the 'd' attribute of the <path> tag.

It should be possible to compress adjacent floating numbers suppressing the leading " 0." of the second float:
"500.13.88" should be parsed equal to "500.13 0.88"

I will include an svg file.
Try open it in Inkscape 0.46, Firefox 3.0 and you will see the correct render.
Nautilus and command line "rsvg" will render miserably...

For reference:
http://www.w3.org/TR/SVG/paths.html#PathDataGeneralInformation
Search for "8.3.9 The grammar for path data"

P.S: I did incorrectly post this bug in 'librsvg' source, https://bugs.launchpad.net/ubuntu/+source/librsvg/+bug/245524
Sorry!

Revision history for this message
Actam (gl.ruggero) wrote :

Bug fixed by gnome team in revision 1167.
http://bugzilla.gnome.org/show_bug.cgi?id=548494

Changed in librsvg:
status: New → Fix Released
Revision history for this message
Pedro Villavicencio (pedro) wrote :

isn't fixed on Ubuntu yet, marking it as fix committed, thanks for pointing to the bug.

Changed in librsvg:
assignee: nobody → desktop-bugs
importance: Undecided → Low
status: Fix Released → Fix Committed
Changed in librsvg:
status: Unknown → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package librsvg - 2.22.3-0ubuntu1

---------------
librsvg (2.22.3-0ubuntu1) intrepid; urgency=low

  * New upstream version:
    - Make librsvg buildable on windows without freetype dependency
    - Fix build and tests
    - Fix the handling of the stroke-dashoffset property
    - Bad rendering for some wacko SVG path data (lp: #259485)
    - Fix some arcs not being well drawn in SVG path
    - Fix various crashes (lp: #196996)
    - Set license field of GdkPixBufFormat in the SVG loader
    - Migrate use of GMemChunk to GSlice
    - If an ID is not found, return an error and/or do not render whole SVG

 -- Sebastien Bacher <email address hidden> Wed, 24 Sep 2008 21:37:56 +0200

Changed in librsvg:
status: Fix Committed → Fix Released
Revision history for this message
Louis Simard (louis-simard-deactivatedaccount) wrote :

Ubuntu Lucid uploaded today a new version of librsvg2-2, 2.26.3-0ubuntu1, which reintroduces this bug as it includes this patch from upstream GNOME:

https://bugzilla.gnome.org/show_bug.cgi^id=563933 "Corrupted rendering of a card in 'Paris' aisleriot card theme"

In the GNOME bug cited above, the patch for this bug is removed in favor of another.

Changed in librsvg (Ubuntu):
status: Fix Released → New
Revision history for this message
Louis Simard (louis-simard-deactivatedaccount) wrote :

Sorry for the bad link. It should be

https://bugzilla.gnome.org/show_bug.cgi?id=563933

tags: added: lucid regression-update
Revision history for this message
Sebastien Bacher (seb128) wrote :

is the bug there fixed or not? if it's fixed but the update has a new issue you should open a new bug rather than reopening one

Changed in librsvg (Ubuntu):
status: New → Incomplete
Revision history for this message
Louis Simard (louis-simard-deactivatedaccount) wrote :

This bug and the one cited in the GNOME bug are mutually exclusive. No fix covers "incorrect float parsing in path data" *and* "corrupt Paris Aisleriot card theme" at once. The upstream bug for "corrupt Paris Aisleriot card theme" (https://bugzilla.gnome.org/show_bug.cgi?id=563933) says that a fix for "incorrect float parsing in path data" (this bug) made it regress, but in doing so, GNOME librsvg reacquired the bug "incorrect float parsing in path data".

Ubuntu's librsvg got the fix for "corrupt Paris Aisleriot card theme" last week. In doing so, this bug reappeared, and is the very same issue that has been reported originally in 2008 and fixed.

For a bug that attempts to fix both the Paris Aisleriot card theme being corrupted, and this bug, please see bug 370061 and mark this one as a duplicate as appropriate.

Revision history for this message
Sebastien Bacher (seb128) wrote :

I'm not sure to understand the summary, so the fix create an another bug and there is no correct fix for both issues yet? The choice is basically between 2 bugs. Which one do you recommend to fix in lucid and which one not and why?

Revision history for this message
Louis Simard (louis-simard-deactivatedaccount) wrote :

Exactly; currently there are fixes in librsvg git for one or the other of these bugs, but not both. The Aisleriot card theme corruption fix... undoes this one before doing its thing.

There *is* a patch proposed in bug 370061's upstream GNOME counterpart (by me, but that's irrelevant) to fix both bugs, as well as anything else that's not to the SVG spec for path parsing in librsvg. But it's not in git yet.

So until then, I would say this bug should be fixed in preference to the Paris Aisleriot card theme corruption bug, because Ubuntu doesn't bundle that card theme.

Revision history for this message
Sebastien Bacher (seb128) wrote :

trading one bug for an another doesn't seem to be the best way, we might want to wait for the upstream bug to be correctly solved

Revision history for this message
Louis Simard (louis-simard-deactivatedaccount) wrote :

Agreed. I'll mark this as a duplicate of bug 370061 the, because that bug is a superset of this one.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.