From 1c617b9f56e7361f11d04ba01a5777b65ca5ad92 Mon Sep 17 00:00:00 2001
From: Seth Hillbrand <1701092@bugs.launchpad.net>
Date: Thu, 31 Aug 2017 12:21:09 -0400
Subject: [PATCH] Prevent duplicate objects during block copy in symbol library
 editor.

Fixes lp:1714109

https://bugs.launchpad.net/kicad/+bug/1714109
---
 eeschema/class_libentry.cpp | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/eeschema/class_libentry.cpp b/eeschema/class_libentry.cpp
index 7b2bbc8863..4ca1e1a9f5 100644
--- a/eeschema/class_libentry.cpp
+++ b/eeschema/class_libentry.cpp
@@ -1491,6 +1491,7 @@ void LIB_PART::CopySelectedItems( const wxPoint& aOffset )
      * a memory reallocation can happen and will break pointers
      */
     unsigned icnt = drawings.size();
+    std::vector< LIB_ITEM* > tmp;
 
     for( unsigned ii = 0; ii < icnt; ii++  )
     {
@@ -1507,9 +1508,12 @@ void LIB_PART::CopySelectedItems( const wxPoint& aOffset )
         item.ClearFlags( SELECTED );
         LIB_ITEM* newItem = (LIB_ITEM*) item.Clone();
         newItem->SetFlags( SELECTED );
-        drawings.push_back( newItem );
+        tmp.push_back( newItem );
     }
 
+    for( unsigned ii = 0;  ii < tmp.size();  ii++ )
+        drawings.push_back( tmp[ii] );
+
     MoveSelectedItems( aOffset );
 }