[libreoffice/f15] fix regression in SvGlobalName operator

Caolan McNamara caolanm at fedoraproject.org
Tue Jul 12 11:04:47 UTC 2011


commit f523b2a6436a716dae38cfbb0ab991d522feeca6
Author: Caolán McNamara <caolanm at redhat.com>
Date:   Tue Jul 12 12:04:35 2011 +0100

    fix regression in SvGlobalName operator

 0001-fix-regression-in-SvGlobalName-operator.patch |   39 ++++++++++++++++++++
 libreoffice.spec                                   |    7 +++-
 2 files changed, 45 insertions(+), 1 deletions(-)
---
diff --git a/0001-fix-regression-in-SvGlobalName-operator.patch b/0001-fix-regression-in-SvGlobalName-operator.patch
new file mode 100644
index 0000000..19c4aeb
--- /dev/null
+++ b/0001-fix-regression-in-SvGlobalName-operator.patch
@@ -0,0 +1,39 @@
+From a935029385159ee137418730363257599f67e5d6 Mon Sep 17 00:00:00 2001
+From: Radek Doulik <rodo at novell.com>
+Date: Mon, 11 Jul 2011 20:36:47 +0200
+Subject: [PATCH] fix regression in SvGlobalName::operator <
+MIME-Version: 1.0
+Content-Type: text/plain; charset=UTF-8
+Content-Transfer-Encoding: 8bit
+
+ - it was comparing wrong parts and ommiting part of ID's, resulting
+   in wrong results and thus ::std::map didn't work well with default
+   less compare function of it's keys
+ - fixes fdo#32709
+(cherry picked from commit 345dc7961bc142f167a1b8e5f43f4439e8234f06)
+
+Signed-off-by: Caolán McNamara <caolanm at redhat.com>
+---
+ tools/source/ref/globname.cxx |    4 ++--
+ 1 files changed, 2 insertions(+), 2 deletions(-)
+
+diff --git a/tools/source/ref/globname.cxx b/tools/source/ref/globname.cxx
+index 7a7f5c5..c78aeb5 100644
+--- a/tools/source/ref/globname.cxx
++++ b/tools/source/ref/globname.cxx
+@@ -208,10 +208,10 @@ BOOL SvGlobalName::operator < ( const SvGlobalName & rObj ) const
+     else if( Data2_a == Data2_b )
+     {
+         sal_uInt32 Data1_a;
+-        memcpy(&Data1_a, pImp->szData+4, sizeof(sal_uInt32));
++        memcpy(&Data1_a, pImp->szData, sizeof(sal_uInt32));
+ 
+         sal_uInt32 Data1_b;
+-        memcpy(&Data1_b, rObj.pImp->szData+4, sizeof(sal_uInt32));
++        memcpy(&Data1_b, rObj.pImp->szData, sizeof(sal_uInt32));
+ 
+         return Data1_a  < Data1_b;
+     }
+-- 
+1.7.6
+
diff --git a/libreoffice.spec b/libreoffice.spec
index 7614512..225451c 100644
--- a/libreoffice.spec
+++ b/libreoffice.spec
@@ -29,7 +29,7 @@ Summary:        Free Software Productivity Suite
 Name:           libreoffice
 Epoch:          1
 Version:        3.3.3.1
-Release:        2%{?dist}
+Release:        3%{?dist}
 License:        LGPLv3 and LGPLv2+ and BSD and (MPLv1.1 or GPLv2 or LGPLv2 or Netscape) and (CDDL or GPLv2) and Public Domain
 Group:          Applications/Productivity
 URL:            http://www.documentfoundation.org/develop
@@ -151,6 +151,7 @@ Patch62: 0001-Related-rhbz-702833-addEventListener-without-removeE.patch
 Patch63: 0001-Related-rhbz-711087-band-aid.patch
 Patch64: 0001-rhbz-667082-do-not-crash-importing-section-containin.patch
 Patch65: 0001-Related-rhbz-718976-crash-in-SwTxtSizeInfo-GetMultiC.patch
+Patch66: 0001-fix-regression-in-SvGlobalName-operator.patch
 
 %{!?python_sitearch: %global python_sitearch %(%{__python} -c "from distutils.sysconfig import get_python_lib; print(get_python_lib(1))")}
 %define instdir %{_libdir}
@@ -817,6 +818,7 @@ mv -f redhat.soc extras/source/palettes/standard.soc
 %patch63 -p1 -b .rhbz711087-band-aid.patch
 %patch64 -p1 -b .rhbz667082-do-not-crash-importing-section-containin.patch
 %patch65 -p1 -b .rhbz718976-crash-in-SwTxtSizeInfo-GetMultiC.patch
+%patch66 -p1 -b .fix-regression-in-SvGlobalName-operator.patch
 
 touch scripting/source/pyprov/delzip
 touch scripting/util/provider/beanshell/delzip
@@ -2161,6 +2163,9 @@ update-desktop-database %{_datadir}/applications &> /dev/null || :
 %{basisinstdir}/program/kde-open-url
 
 %changelog
+* Tue Jul 12 2011 Caolán McNamara <caolanm at redhat.com> 1:3.3.3.1-3-UNBUILT
+- fix regression in SvGlobalName operator
+
 * Tue Jul 05 2011 Caolán McNamara <caolanm at redhat.com> 1:3.3.3.1-2
 - Resolves: rhbz#713154 pdf export dialog too tall to fit
 - Related: rhbz#702833 addEventListener without removeEventListener


More information about the scm-commits mailing list