From 279ccd4fe28145642bb11dc8dcf0c5a9cdcbf9da Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Kristoffer=20=C3=96dmark?= <kristoffer.odmark90@gmail.com>
Date: Tue, 19 Sep 2017 01:07:28 +0200
Subject: [PATCH] Fix bug found by Tomasz, added constant modifier in loops
 where board items should not be modified

---
 pcbnew/kicad_clipboard.cpp | 26 ++++++++++++++------------
 1 file changed, 14 insertions(+), 12 deletions(-)

diff --git a/pcbnew/kicad_clipboard.cpp b/pcbnew/kicad_clipboard.cpp
index 4d5fae60a3..336ffe78bf 100644
--- a/pcbnew/kicad_clipboard.cpp
+++ b/pcbnew/kicad_clipboard.cpp
@@ -71,7 +71,7 @@ void CLIPBOARD_IO::SaveSelection( const SELECTION& aSelected )
 
     // Differentiate how it is formatted depending on what selection contains
     bool onlyModuleParts = true;
-    for( auto i : aSelected )
+    for( const auto i : aSelected )
     {
         // check if it not one of the module primitives
         if( ( i->Type() != PCB_MODULE_EDGE_T ) &&
@@ -85,25 +85,27 @@ void CLIPBOARD_IO::SaveSelection( const SELECTION& aSelected )
 
     // if there is only parts of a module selected, format it as a new module else
     // format it as an entire board
-    MODULE module( m_board );
+    MODULE partialModule( m_board );
 
     // only a module selected.
     if( aSelected.Size() == 1 && aSelected.Front()->Type() == PCB_MODULE_T )
     {
         // make the module safe to transfer to other pcbs
-        MODULE* mod = static_cast<MODULE*>( aSelected.Front() );
-        for( D_PAD* pad = mod->PadsList().begin(); pad; pad = pad->Next() )
+        const MODULE* mod = static_cast<MODULE*>( aSelected.Front() );
+        // Do not modify existing board
+        MODULE newModule(*mod);
+
+        for( D_PAD* pad = newModule.PadsList().begin(); pad; pad = pad->Next() )
         {
             pad->SetNetCode( 0,0 );
         }
 
-        Format(static_cast<BOARD_ITEM*>( mod ) );
+        Format( static_cast<BOARD_ITEM*>( &newModule ) );
     }
-
     // partial module selected.
     else if( onlyModuleParts )
     {
-        for( auto item : aSelected )
+        for( const auto item : aSelected )
         {
             auto clone = static_cast<BOARD_ITEM*>( item->Clone() );
 
@@ -118,16 +120,16 @@ void CLIPBOARD_IO::SaveSelection( const SELECTION& aSelected )
                pad->SetNetCode( 0, 0 );
             }
 
-            module.Add( clone );
+            partialModule.Add( clone );
         }
 
         // Set the new relative internal local coordinates of copied items
         MODULE* editedModule = m_board->m_Modules;
-        wxPoint moveVector = module.GetPosition() + editedModule->GetPosition();
+        wxPoint moveVector = partialModule.GetPosition() + editedModule->GetPosition();
 
-        module.MoveAnchorPosition( moveVector );
+        partialModule.MoveAnchorPosition( moveVector );
 
-        Format( &module, 0 );
+        Format( &partialModule, 0 );
 
     }
     // lots of stuff selected
@@ -147,7 +149,7 @@ void CLIPBOARD_IO::SaveSelection( const SELECTION& aSelected )
         m_formatter.Print( 0, "\n" );
 
 
-        for( auto i : aSelected )
+        for( const auto i : aSelected )
         {
             // Dont format stuff that cannot exist standalone!
             if( ( i->Type() != PCB_MODULE_EDGE_T ) &&