kicad-developers team mailing list archive

[Jean-Samuel Reynaud <js.reynaud@xxxxxxxxx>]

Hi all,

There is an incorrect initialisation of variable m_cur_grid in
DIALOG_SYMBOL_LIB_TABLE (file eeschema/dialogs/dialog_sym_lib_table.cpp).

This result of a crash on eeschema when:
- opening "Symbol Library Table"
- click on "Ok" button

If before clicking on "Ok" you switch between "Global libraries" and
"Project Specific Librairies", the tab switch set correctly m_cur_grid
variable (with the handler pageChangedHandler) and the bug is no more
reproductible.

Please find attached the patch to fix this.


Regards,

From 656814635fcf45480420c8bfc0ad3c09052b642f Mon Sep 17 00:00:00 2001
From: Jean-Samuel Reynaud <js.reynaud@xxxxxxxxx>
Date: Wed, 30 Aug 2017 19:36:03 +0200
Subject: [PATCH] Fix incorrect (missing) initialisation of m_cur_grid in
 symbols libraries table dialog box
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="------------2.7.4"

This is a multi-part message in MIME format.
--------------2.7.4
Content-Type: text/plain; charset=UTF-8; format=fixed
Content-Transfer-Encoding: 8bit

---
 eeschema/dialogs/dialog_sym_lib_table.cpp | 6 ++++++
 1 file changed, 6 insertions(+)


--------------2.7.4
Content-Type: text/x-patch; name="0001-Fix-incorrect-missing-initialisation-of-m_cur_grid-i.patch"
Content-Transfer-Encoding: 8bit
Content-Disposition: attachment; filename="0001-Fix-incorrect-missing-initialisation-of-m_cur_grid-i.patch"

diff --git a/eeschema/dialogs/dialog_sym_lib_table.cpp b/eeschema/dialogs/dialog_sym_lib_table.cpp
index ccaead2..cf091e5 100644
--- a/eeschema/dialogs/dialog_sym_lib_table.cpp
+++ b/eeschema/dialogs/dialog_sym_lib_table.cpp
@@ -202,9 +202,15 @@ DIALOG_SYMBOL_LIB_TABLE::DIALOG_SYMBOL_LIB_TABLE( wxTopLevelWindow* aParent,
 
     // for ALT+A handling, we want the initial focus to be on the first selected grid.
     if( m_pageNdx == 0 )
+    {
         m_global_grid->SetFocus();
+        m_cur_grid = m_global_grid;
+    }
     else
+    {
         m_project_grid->SetFocus();
+        m_cur_grid = m_project_grid;
+    }
 
     // On some window managers (Unity, XFCE), this dialog is
     // not always raised, depending on this dialog is run.

--------------2.7.4--