[Libreoffice-commits] core.git: Branch 'libreoffice-5-1' - external/mdds

Caolán McNamara caolanm at redhat.com
Wed Jun 22 10:35:31 UTC 2016


 external/mdds/UnpackedTarball_mdds.mk |    1 +
 external/mdds/tdf90579.patch.0        |   23 +++++++++++++++++++++++
 2 files changed, 24 insertions(+)

New commits:
commit 19312e13c4ab802c2bd878f4448556f245884a13
Author: Caolán McNamara <caolanm at redhat.com>
Date:   Fri Jun 17 14:37:17 2016 +0100

    Resolves: tdf#90579 swap_single_to_multi_blocks is broken
    
    when there is associated data in the src. Its copies those pointers
    into the block that will appear in the destination. Then can destroy
    the source block, which deletes the contents of those pointers, and
    then inserts the new block which has dangling pointers to the deleted
    data.
    
    https://gitlab.com/mdds/mdds/merge_requests/2
    
    (cherry picked from commit 9ec54e92407cd632c4e38317f914edd557835a86)
    (cherry picked from commit 6114072a59cfff36218aea70e1b52fa4c3ba64b4)
    
    Change-Id: Id9614d95652c8032b03cb5748a284917043d8d21
    Reviewed-on: https://gerrit.libreoffice.org/26429
    Tested-by: Jenkins <ci at libreoffice.org>
    Reviewed-by: David Tardon <dtardon at redhat.com>

diff --git a/external/mdds/UnpackedTarball_mdds.mk b/external/mdds/UnpackedTarball_mdds.mk
index 504406b..1ccd5de 100644
--- a/external/mdds/UnpackedTarball_mdds.mk
+++ b/external/mdds/UnpackedTarball_mdds.mk
@@ -17,6 +17,7 @@ $(eval $(call gb_UnpackedTarball_add_patches,mdds,\
 	external/mdds/mdds_0.6.0.patch \
 	external/mdds/mdds-c++98.patch.0 \
 	external/mdds/0001-another-step-to-allow-fst-to-use-any-value-type.patch.1 \
+	external/mdds/tdf90579.patch.0 \
 ))
 
 # vim: set noet sw=4 ts=4:
diff --git a/external/mdds/tdf90579.patch.0 b/external/mdds/tdf90579.patch.0
new file mode 100644
index 0000000..5ef1bc7
--- /dev/null
+++ b/external/mdds/tdf90579.patch.0
@@ -0,0 +1,23 @@
+diff --git a/include/mdds/multi_type_vector_def.inl b/include/mdds/multi_type_vector_def.inl
+index 0e2a15a..fe9c767 100644
+--- include/mdds/multi_type_vector_def.inl
++++ include/mdds/multi_type_vector_def.inl
+@@ -2306,6 +2306,9 @@ void multi_type_vector<_CellBlockFunc, _EventFunc>::swap_single_to_multi_blocks(
+     {
+         // Source range is at the top of a block.
+ 
++        // Shrink the current block by erasing the top part.
++        element_block_func::erase(*blk_src->mp_data, 0, len);
++
+         if (src_tail_len == 0)
+         {
+             // the whole block needs to be replaced.
+@@ -2314,8 +2317,6 @@ void multi_type_vector<_CellBlockFunc, _EventFunc>::swap_single_to_multi_blocks(
+         }
+         else
+         {
+-            // Shrink the current block by erasing the top part.
+-            element_block_func::erase(*blk_src->mp_data, 0, len);
+             blk_src->m_size -= len;
+         }
+ 


More information about the Libreoffice-commits mailing list