[sisyphus] inkscape/poppler crashes

[sisyphus] inkscape/poppler crashes

[Ivan Adzhubey]

Hi all,

I am having a problem with the latest Sisyphus Inkscape version that looks 
identical with the one described via the link below:

http://old.nabble.com/poppler-0.12.1-vs-poppler-0.12.%282|
3%29-ABI-%28and-API%29-breakage--td27179664.html

Inkscape crashes upon importing certain ps/eps/pdf files with the following 
error message:

inkscape: symbol lookup error: inkscape: undefined symbol: 
_ZN13GfxColorSpace5parseEP6Object

$ rpm -qa|fgrep inkscape
inkscape-0.47-alt4

$ rpm -qa|fgrep poppler|sort
libpoppler08-0.8.7-alt2
libpoppler08-glib-0.8.7-alt2
libpoppler2-qt-0.12.3-alt1
libpoppler3-qt4-0.12.3-alt1
libpoppler4-0.10.7-alt2
libpoppler4-glib-0.12.3-alt1
libpoppler5-0.12.3-alt1
poppler-0.12.3-alt1
poppler-data-0.4.0-alt1

$ uname -m
x86_64

--Ivan


The information in this e-mail is intended only for the person to whom it is
addressed. If you believe this e-mail was sent to you in error and the e-mail
contains patient information, please contact the Partners Compliance HelpLine at
http://www.partners.org/complianceline . If the e-mail was sent to you in error
but does not contain patient information, please contact the sender and properly
dispose of the e-mail.

Re: [sisyphus] libpoppler.so.5 backwards incompatibility

[Dmitry V. Levin]

[-- Attachment #1: Type: text/plain, Size: 1049 bytes --]

Hi,

On Thu, Jan 21, 2010 at 04:45:51PM -0500, Ivan Adzhubey wrote:
[...]
> Inkscape crashes upon importing certain ps/eps/pdf files with the following 
> error message:

Ivan, thank you for pointing it out.

> inkscape: symbol lookup error: inkscape: undefined symbol: 
> _ZN13GfxColorSpace5parseEP6Object

I'd be surprised if it didn't:

$ ldd -r /usr/bin/inkscape 2>&1 >/dev/null |c++filt 
undefined symbol: GfxResources::lookupShading(char*) (/usr/bin/inkscape)
undefined symbol: GfxColorSpace::parse(Object*)	(/usr/bin/inkscape)
undefined symbol: GfxResources::lookupPattern(char*) (/usr/bin/inkscape)

That happens because libpoppler.so.5 failed to provide backwards
compatibility.

> $ rpm -qa|fgrep inkscape
> inkscape-0.47-alt4

inkview suffers from the same problem.

Simple rebuild of inkscape should cure the crash, but not the problem in
general because libpoppler changes ABI incompatibly from time to time.

2zerg: You have to change soname every time such things happen with
libpoppler.


-- 
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 198 bytes --]

Re: [sisyphus] libpoppler.so.5 backwards incompatibility

[Ivan Adzhubey]

Dmitry, spasibo chto nashli vremja vzgljanut! Est' li shans na 
rabotosposobnuju sborku? Google pokazyvaet chto problema obsuzhdaetsa no 
gotovogo reshenija ja ne vizhu.

Proshu proshenija za English i za translit, russkogo na rabote na klaviature 
net. No chitat' Cyrillic ja mogu tak chto ne objazatel'no mne otvechat' 
po-anglijski ;-)

--Ivan

On Thursday 21 January 2010 05:53:37 pm Dmitry V. Levin wrote:
> Hi,
>
> On Thu, Jan 21, 2010 at 04:45:51PM -0500, Ivan Adzhubey wrote:
> [...]
>
> > Inkscape crashes upon importing certain ps/eps/pdf files with the
> > following error message:
>
> Ivan, thank you for pointing it out.
>
> > inkscape: symbol lookup error: inkscape: undefined symbol:
> > _ZN13GfxColorSpace5parseEP6Object
>
> I'd be surprised if it didn't:
>
> $ ldd -r /usr/bin/inkscape 2>&1 >/dev/null |c++filt
> undefined symbol: GfxResources::lookupShading(char*) (/usr/bin/inkscape)
> undefined symbol: GfxColorSpace::parse(Object*)	(/usr/bin/inkscape)
> undefined symbol: GfxResources::lookupPattern(char*) (/usr/bin/inkscape)
>
> That happens because libpoppler.so.5 failed to provide backwards
> compatibility.
>
> > $ rpm -qa|fgrep inkscape
> > inkscape-0.47-alt4
>
> inkview suffers from the same problem.
>
> Simple rebuild of inkscape should cure the crash, but not the problem in
> general because libpoppler changes ABI incompatibly from time to time.
>
> 2zerg: You have to change soname every time such things happen with
> libpoppler.




The information in this e-mail is intended only for the person to whom it is
addressed. If you believe this e-mail was sent to you in error and the e-mail
contains patient information, please contact the Partners Compliance HelpLine at
http://www.partners.org/complianceline . If the e-mail was sent to you in error
but does not contain patient information, please contact the sender and properly
dispose of the e-mail.

Re: [sisyphus] libpoppler.so.5 backwards incompatibility

[Dmitry V. Levin]

[-- Attachment #1: Type: text/plain, Size: 1144 bytes --]

On Thu, Jan 21, 2010 at 06:15:57PM -0500, Ivan Adzhubey wrote:
> Dmitry, spasibo chto nashli vremja vzgljanut! Est' li shans na 
> rabotosposobnuju sborku? Google pokazyvaet chto problema obsuzhdaetsa no 
> gotovogo reshenija ja ne vizhu.
> 
> Proshu proshenija za English i za translit, russkogo na rabote na klaviature 
> net. No chitat' Cyrillic ja mogu tak chto ne objazatel'no mne otvechat' 
> po-anglijski ;-)

Я не специалист ни в poppler, ни в inkscape.  Пакет inkscape-0.47-alt4 в
Сизифе перестал проходить тест на пересборку сразу после того, как в
Сизифе обновился пакет poppler5 с версии 0.12.1 на более новую:

extension/internal/pdfinput/pdf-parser.cpp: In member function 'void PdfParser::opSetExtGState(Object*, int)':
extension/internal/pdfinput/pdf-parser.cpp:812: error: no matching function for call to 'GfxColorSpace::parse(Object*)'
/usr/include/poppler/GfxState.h:182: note: candidates are: static GfxColorSpace* GfxColorSpace::parse(Object*, Gfx*)
(и ещё куча аналогичных ошибок из-за смены API у poppler'а)

Я надеюсь, что upstream poppler'а это скоро исправит (если ещё не
исправил).


-- 
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 198 bytes --]

Re: [sisyphus] libpoppler.so.5 backwards incompatibility

[Dmitry V. Levin]

[-- Attachment #1.1: Type: text/plain, Size: 1360 bytes --]

On Fri, Jan 22, 2010 at 02:35:25AM +0300, Dmitry V. Levin wrote:
> On Thu, Jan 21, 2010 at 06:15:57PM -0500, Ivan Adzhubey wrote:
> > Dmitry, spasibo chto nashli vremja vzgljanut! Est' li shans na 
> > rabotosposobnuju sborku? Google pokazyvaet chto problema obsuzhdaetsa no 
> > gotovogo reshenija ja ne vizhu.
> > 
> > Proshu proshenija za English i za translit, russkogo na rabote na klaviature 
> > net. No chitat' Cyrillic ja mogu tak chto ne objazatel'no mne otvechat' 
> > po-anglijski ;-)
> 
> Я не специалист ни в poppler, ни в inkscape.  Пакет inkscape-0.47-alt4 в
> Сизифе перестал проходить тест на пересборку сразу после того, как в
> Сизифе обновился пакет poppler5 с версии 0.12.1 на более новую:
> 
> extension/internal/pdfinput/pdf-parser.cpp: In member function 'void PdfParser::opSetExtGState(Object*, int)':
> extension/internal/pdfinput/pdf-parser.cpp:812: error: no matching function for call to 'GfxColorSpace::parse(Object*)'
> /usr/include/poppler/GfxState.h:182: note: candidates are: static GfxColorSpace* GfxColorSpace::parse(Object*, Gfx*)
> (и ещё куча аналогичных ошибок из-за смены API у poppler'а)
> 
> Я надеюсь, что upstream poppler'а это скоро исправит (если ещё не
> исправил).

Похоже что уже исправили, см.
https://bugs.launchpad.net/inkscape/+bug/487038
и приложенный патч оттуда.


-- 
ldv

[-- Attachment #1.2: inkscape-ubuntu-poppler-API.patch --]
[-- Type: text/plain, Size: 3674 bytes --]

#! /bin/sh /usr/share/dpatch/dpatch-run
## 50-poppler-API.patch.dpatch by Kees Cook <kees@ubuntu.com>
##
## All lines beginning with `## DP:' are a description of the patch.
## DP: https://bugs.launchpad.net/inkscape/+bug/487038

@DPATCH@
diff -urNad inkscape~/src/extension/internal/pdfinput/pdf-parser.cpp inkscape/src/extension/internal/pdfinput/pdf-parser.cpp
--- inkscape~/src/extension/internal/pdfinput/pdf-parser.cpp	2009-10-10 12:17:47.000000000 -0700
+++ inkscape/src/extension/internal/pdfinput/pdf-parser.cpp	2009-12-23 22:34:13.000000000 -0800
@@ -809,7 +809,7 @@
 	  blendingColorSpace = NULL;
 	  isolated = knockout = gFalse;
 	  if (!obj4.dictLookup(const_cast<char*>("CS"), &obj5)->isNull()) {
-	    blendingColorSpace = GfxColorSpace::parse(&obj5);
+	    blendingColorSpace = GfxColorSpace::parse(&obj5, NULL);
 	  }
 	  obj5.free();
 	  if (obj4.dictLookup(const_cast<char*>("I"), &obj5)->isBool()) {
@@ -1009,9 +1009,9 @@
   state->setFillPattern(NULL);
   res->lookupColorSpace(args[0].getName(), &obj);
   if (obj.isNull()) {
-    colorSpace = GfxColorSpace::parse(&args[0]);
+    colorSpace = GfxColorSpace::parse(&args[0], NULL);
   } else {
-    colorSpace = GfxColorSpace::parse(&obj);
+    colorSpace = GfxColorSpace::parse(&obj, NULL);
   }
   obj.free();
   if (colorSpace) {
@@ -1032,9 +1032,9 @@
   state->setStrokePattern(NULL);
   res->lookupColorSpace(args[0].getName(), &obj);
   if (obj.isNull()) {
-    colorSpace = GfxColorSpace::parse(&args[0]);
+    colorSpace = GfxColorSpace::parse(&args[0], NULL);
   } else {
-    colorSpace = GfxColorSpace::parse(&obj);
+    colorSpace = GfxColorSpace::parse(&obj, NULL);
   }
   obj.free();
   if (colorSpace) {
@@ -1101,7 +1101,7 @@
       builder->updateStyle(state);
     }
     if (args[numArgs-1].isName() &&
-	(pattern = res->lookupPattern(args[numArgs-1].getName()))) {
+	(pattern = res->lookupPattern(args[numArgs-1].getName(), NULL))) {
       state->setFillPattern(pattern);
       builder->updateStyle(state);
     }
@@ -1145,7 +1145,7 @@
       builder->updateStyle(state);
     }
     if (args[numArgs-1].isName() &&
-	(pattern = res->lookupPattern(args[numArgs-1].getName()))) {
+	(pattern = res->lookupPattern(args[numArgs-1].getName(), NULL))) {
       state->setStrokePattern(pattern);
       builder->updateStyle(state);
     }
@@ -1543,7 +1543,7 @@
   double *matrix = NULL;
   GBool savedState = gFalse;
 
-  if (!(shading = res->lookupShading(args[0].getName()))) {
+  if (!(shading = res->lookupShading(args[0].getName(), NULL))) {
     return;
   }
 
@@ -2507,7 +2507,7 @@
       }
     }
     if (!obj1.isNull()) {
-      colorSpace = GfxColorSpace::parse(&obj1);
+      colorSpace = GfxColorSpace::parse(&obj1, NULL);
     } else if (csMode == streamCSDeviceGray) {
       colorSpace = new GfxDeviceGrayColorSpace();
     } else if (csMode == streamCSDeviceRGB) {
@@ -2592,7 +2592,7 @@
 	  obj2.free();
 	}
       }
-      maskColorSpace = GfxColorSpace::parse(&obj1);
+      maskColorSpace = GfxColorSpace::parse(&obj1, NULL);
       obj1.free();
       if (!maskColorSpace || maskColorSpace->getMode() != csDeviceGray) {
 	goto err1;
@@ -2767,7 +2767,7 @@
     if (obj1.dictLookup(const_cast<char*>("S"), &obj2)->isName(const_cast<char*>("Transparency"))) {
       transpGroup = gTrue;
       if (!obj1.dictLookup(const_cast<char*>("CS"), &obj3)->isNull()) {
-	blendingColorSpace = GfxColorSpace::parse(&obj3);
+	blendingColorSpace = GfxColorSpace::parse(&obj3, NULL);
       }
       obj3.free();
       if (obj1.dictLookup(const_cast<char*>("I"), &obj3)->isBool()) {

[-- Attachment #2: Type: application/pgp-signature, Size: 198 bytes --]

Re: [sisyphus] libpoppler.so.5 backwards incompatibility

[Ivan Adzhubey]

On Thursday 21 January 2010 06:56:05 pm Dmitry V. Levin wrote:
> On Fri, Jan 22, 2010 at 02:35:25AM +0300, Dmitry V. Levin wrote:
> > On Thu, Jan 21, 2010 at 06:15:57PM -0500, Ivan Adzhubey wrote:
> > > Dmitry, spasibo chto nashli vremja vzgljanut! Est' li shans na
> > > rabotosposobnuju sborku? Google pokazyvaet chto problema obsuzhdaetsa
> > > no gotovogo reshenija ja ne vizhu.
> > >
> > > Proshu proshenija za English i za translit, russkogo na rabote na
> > > klaviature net. No chitat' Cyrillic ja mogu tak chto ne objazatel'no
> > > mne otvechat' po-anglijski ;-)
> >
> > Я не специалист ни в poppler, ни в inkscape.  Пакет inkscape-0.47-alt4 в
> > Сизифе перестал проходить тест на пересборку сразу после того, как в
> > Сизифе обновился пакет poppler5 с версии 0.12.1 на более новую:
> >
> > extension/internal/pdfinput/pdf-parser.cpp: In member function 'void
> > PdfParser::opSetExtGState(Object*, int)':
> > extension/internal/pdfinput/pdf-parser.cpp:812: error: no matching
> > function for call to 'GfxColorSpace::parse(Object*)'
> > /usr/include/poppler/GfxState.h:182: note: candidates are: static
> > GfxColorSpace* GfxColorSpace::parse(Object*, Gfx*) (и ещё куча
> > аналогичных ошибок из-за смены API у poppler'а)
> >
> > Я надеюсь, что upstream poppler'а это скоро исправит (если ещё не
> > исправил).
> 
> Похоже что уже исправили, см.
> https://bugs.launchpad.net/inkscape/+bug/487038
> и приложенный патч оттуда.

Дома на Ubuntu 9.10 inkscape на тех же файлах не падает, но poppler5 в Ubuntu 
старше:

libpoppler5                          0.12.0-0ubuntu2.1
inkscape                             0.47~pre4-0ubuntu1

--Иван

Re: [sisyphus] libpoppler.so.5 backwards incompatibility

[Sergei Epiphanov]

В сообщении от 22 января 2010 06:30:06 автор Ivan Adzhubey написал:
> inkscape                             0.47~pre4-0ubuntu1

Интересная запись версии. Так используется в Ubuntu? 

-- 
С уважением, Епифанов Сергей

Re: [sisyphus] libpoppler.so.5 backwards incompatibility

[Andrey Rahmatullin]

[-- Attachment #1: Type: text/plain, Size: 408 bytes --]

On Fri, Jan 22, 2010 at 02:20:17PM +0300, Sergei Epiphanov wrote:
> > inkscape                             0.47~pre4-0ubuntu1
> Интересная запись версии. Так используется в Ubuntu? 
В Debian. Насколько я помню, 0.46 < 0.47~чтоугодно < 0.47

-- 
WBR, wRAR (ALT Linux Team)
Powered by the ALT Linux fortune(6):

Винда может быть надёжной при одном условии -- power off.
		-- mithraen in smoke-room@

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 490 bytes --]

Re: [sisyphus] libpoppler.so.5 backwards incompatibility

[Sergey V Turchin]

[-- Attachment #1: Type: Text/Plain, Size: 472 bytes --]

On Friday 22 January 2010, Dmitry V. Levin wrote:
> Hi,
> 
> On Thu, Jan 21, 2010 at 04:45:51PM -0500, Ivan Adzhubey wrote:

[...]

> 2zerg: You have to change soname every time such things happen
>  with libpoppler.
Программы используют неподдерживеамые(могу запаковать отдельно, чтоб 
увидеть, кто отвалиться) заголовки от xpdf на свой страх и риск.

-- 
Regards, Sergey, ALT Linux, http://www.altlinux.ru/
http://stinkfoot.org:11371/pks/lookup?op=get&search=0x1C2A3F08

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

Re: [sisyphus] libpoppler.so.5 backwards incompatibility

[Sergey V Turchin]

[-- Attachment #1: Type: Text/Plain, Size: 376 bytes --]

On Friday 22 January 2010, Dmitry V. Levin wrote:

[...]
> Похоже что уже исправили, см.
> https://bugs.launchpad.net/inkscape/+bug/487038
> и приложенный патч оттуда.
Да,
https://launchpad.net/ubuntu/+source/inkscape/0.47.0-1ubuntu2
собран с таким патчем

-- 
Regards, Sergey, ALT Linux, http://www.altlinux.ru/
http://stinkfoot.org:11371/pks/lookup?op=get&search=0x1C2A3F08

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 198 bytes --]