[KPhotoAlbum] crash when calling external tools

[vincent delft]

Great!!! it works!!!!

I've used this patch on kpa-3.0.2.

Thanks

For your info, here after libraries I'm using (after
the patch and recompilation):
"
[vi at myhost ~]$ ldd /opt/kde/bin/kphotoalbum
        linux-gate.so.1 =>  (0xb7ef9000)
        libjpeg.so.62 => /usr/lib/libjpeg.so.62
(0xb7eba000)
        libkipi.so.0 => /opt/kde/lib/libkipi.so.0
(0xb7e86000)
        libexiv2.so.0 => /usr/lib/libexiv2.so.0
(0xb7d8f000)
        libkmediaplayer.so.0 =>
/opt/kde/lib/libkmediaplayer.so.0 (0xb7d84000)
        libkparts.so.2 => /opt/kde/lib/libkparts.so.2
(0xb7d45000)
        libkio.so.4 => /opt/kde/lib/libkio.so.4
(0xb7a3a000)
        libkdeui.so.4 => /opt/kde/lib/libkdeui.so.4
(0xb7777000)
        libkdesu.so.4 => /opt/kde/lib/libkdesu.so.4
(0xb7761000)
        libkwalletclient.so.1 =>
/opt/kde/lib/libkwalletclient.so.1 (0xb7751000)
        libkdecore.so.4 =>
/opt/kde/lib/libkdecore.so.4 (0xb7542000)
        libDCOP.so.4 => /opt/kde/lib/libDCOP.so.4
(0xb7514000)
        libdl.so.2 => /lib/libdl.so.2 (0xb750f000)
        libresolv.so.2 => /lib/libresolv.so.2
(0xb74fd000)
        libutil.so.1 => /lib/libutil.so.1 (0xb74f9000)
        libart_lgpl_2.so.2 =>
/usr/lib/libart_lgpl_2.so.2 (0xb74e4000)
        libidn.so.11 => /usr/lib/libidn.so.11
(0xb74b3000)
        libkdefx.so.4 => /opt/kde/lib/libkdefx.so.4
(0xb748b000)
        libqt-mt.so.3 => /opt/qt/lib/libqt-mt.so.3
(0xb6da1000)
        libpng12.so.0 => /usr/lib/libpng12.so.0
(0xb6d7d000)
        libXext.so.6 => /usr/lib/libXext.so.6
(0xb6d6f000)
        libX11.so.6 => /usr/lib/libX11.so.6
(0xb6c86000)
        libSM.so.6 => /usr/lib/libSM.so.6 (0xb6c7e000)
        libICE.so.6 => /usr/lib/libICE.so.6
(0xb6c67000)
        libpthread.so.0 => /lib/libpthread.so.0
(0xb6c4f000)
        libXrender.so.1 => /usr/lib/libXrender.so.1
(0xb6c46000)
        libz.so.1 => /lib/libz.so.1 (0xb6c34000)
        libfam.so.0 => /usr/lib/libfam.so.0
(0xb6c2d000)
        libacl.so.1 => /lib/libacl.so.1 (0xb6c26000)
        libattr.so.1 => /lib/libattr.so.1 (0xb6c22000)
        libstdc++.so.6 => /usr/lib/libstdc++.so.6
(0xb6b34000)
        libm.so.6 => /lib/libm.so.6 (0xb6b0e000)
        libgcc_s.so.1 => /usr/lib/libgcc_s.so.1
(0xb6b03000)
        libc.so.6 => /lib/libc.so.6 (0xb69cd000)
        /lib/ld-linux.so.2 (0xb7efa000)
        libGL.so.1 => /usr/lib/libGL.so.1 (0xb696f000)
        libXmu.so.6 => /usr/lib/libXmu.so.6
(0xb695a000)
        libXrandr.so.2 => /usr/lib/libXrandr.so.2
(0xb6954000)
        libXcursor.so.1 => /usr/lib/libXcursor.so.1
(0xb694b000)
        libXinerama.so.1 => /usr/lib/libXinerama.so.1
(0xb6948000)
        libXft.so.2 => /usr/lib/libXft.so.2
(0xb6936000)
        libfreetype.so.6 => /usr/lib/libfreetype.so.6
(0xb68ba000)
        libfontconfig.so.1 =>
/usr/lib/libfontconfig.so.1 (0xb688f000)
        libXau.so.6 => /usr/lib/libXau.so.6
(0xb688c000)
        libxcb-xlib.so.0 => /usr/lib/libxcb-xlib.so.0
(0xb688a000)
        libxcb.so.1 => /usr/lib/libxcb.so.1
(0xb6872000)
        libXxf86vm.so.1 => /usr/lib/libXxf86vm.so.1
(0xb686c000)
        libXdamage.so.1 => /usr/lib/libXdamage.so.1
(0xb6869000)
        libXfixes.so.3 => /usr/lib/libXfixes.so.3
(0xb6864000)
        libdrm.so.2 => /usr/lib/libdrm.so.2
(0xb685b000)
        libXt.so.6 => /usr/lib/libXt.so.6 (0xb680c000)
        libexpat.so.1 => /usr/lib/libexpat.so.1
(0xb67eb000)
        libXdmcp.so.6 => /usr/lib/libXdmcp.so.6
(0xb67e6000)
[vi at myhost ~]$                                        
            
"




--- Jan Kundrát <jkt at gentoo.org> wrote:

> Jan Kundrát wrote:
> > Qt mangled the name with "&" hotkey. It's fixed as
> of r736328. If you
> > don't use SVN, the patch is attached. Thanks for
> reporting.
> 
> Or rather r736331 and this one :), sorry.
> 
> Cheers,
> -jkt
> 
> -- 
> cd /local/pub && more beer > /dev/mouth
> > Index: MainWindow/ExternalPopup.cpp
>
===================================================================
> --- MainWindow/ExternalPopup.cpp	(revision 733678)
> +++ MainWindow/ExternalPopup.cpp	(revision 736331)
> @@ -69,7 +69,7 @@
>  
>  void MainWindow::ExternalPopup::slotExecuteService(
> int id )
>  {
> -    QString name = text( id );
> +    QString name = text( id ).remove('&');
>      KTrader::OfferList offers =
> KTrader::self()->query(
> *(_appToMimeTypeMap[name].begin()),
> QString::fromLatin1("Type == 'Application' and Name
> == '%1'").arg(name));
>      Q_ASSERT( offers.count() == 1 );
>      KService::Ptr ptr = offers.first();
> @@ -115,7 +115,7 @@
>          KTrader::OfferList offers =
> KTrader::self()->query( *mimeTypeIt,
> QString::fromLatin1("Type == 'Application'"));
>          for(KTrader::OfferList::Iterator offerIt =
> offers.begin(); offerIt != offers.end(); ++offerIt)
> {
>              res.insert( qMakePair(
> (*offerIt)->name(),
> (*offerIt)->pixmap(KIcon::Toolbar) ) );
> -           
> _appToMimeTypeMap[(*offerIt)->name()].insert(
> *mimeTypeIt );
> +           
>
_appToMimeTypeMap[(*offerIt)->name().remove('&')].insert(
> *mimeTypeIt );
>          }
>      }
>      return res;
> > _______________________________________________
> KPhotoAlbum mailing list
> KPhotoAlbum at kdab.net
> http://mail.kdab.net/mailman/listinfo/kphotoalbum
> 



      ____________________________________________________________________________________
Be a better sports nut!  Let your teams follow you 
with Yahoo Mobile. Try it now.  http://mobile.yahoo.com/sports;_ylt=At9_qDKvtAbMuh1G1SQtBI7ntAcJ