Skip to content
Snippets Groups Projects
New look: Off

Kicad segfault on missing color scheme

    • Closed Issue created by Niki Guldbrand @Niki.Guldbrand

    Description

    I was playing around with color scheme in connection with #5030 (closed), which resulted in a segfault.

    WX Backtrace:

    ASSERT INFO:
../src/common/ctrlsub.cpp(231): assert "IsValid(n)" failed in GetClientData(): Invalid index passed to GetClientData()

BACKTRACE:
[1] wxItemContainer::GetClientData(unsigned int) const
[2] wxEvtHandler::ProcessEventIfMatchesId(wxEventTableEntryBase const&, wxEvtHandler*, wxEvent&)
[3] wxEventHashTable::HandleEvent(wxEvent&, wxEvtHandler*)
[4] wxEvtHandler::TryHereOnly(wxEvent&)
[5] wxEvtHandler::DoTryChain(wxEvent&)
[6] wxEvtHandler::ProcessEvent(wxEvent&)
[7] wxWindowBase::TryAfter(wxEvent&)
[8] wxEvtHandler::SafelyProcessEvent(wxEvent&)
[9] wxMenuBase::SendEvent(int, int)
[10] g_closure_invoke
[11] g_signal_emit_valist
[12] g_signal_emit
[13] gtk_widget_activate
[14] gtk_menu_shell_activate_item
[15] g_signal_emit_valist
[16] g_signal_emit
[17] gtk_main_do_event
[18] g_main_context_dispatch
[19] g_main_loop_run
[20] gtk_main
[21] wxGUIEventLoop::DoRun()
[22] wxEventLoopBase::Run()
[23] wxAppConsoleBase::MainLoop()
[24] APP_KICAD::OnRun() /home/niki/Sync/Kicad/Software/kicad-master/kicad/kicad.cpp:279
[25] wxEntry(int&, wchar_t**)
[26] main /home/niki/Sync/Kicad/Software/kicad-master/kicad/kicad.cpp:312
[27] __libc_start_main
[28] _start

    bt_full.txt

    Steps to reproduce

    1. Create a new color scheme or use an existing none default one, and make it the default.
    2. Close Kicad.
    3. Go into ${HOME}/.config/kicad/colors and rename the currently set color scheme.
    4. Start Kicad and start eeschema.
    5. Try to open Preferences in the preferences menu, crash.
    6. Rename the color scheme file back to it's original file name.
    7. Start Kicad, Eeschema, try to go to the preferences and the window opens

    KiCad Version

    Marked dirty because of #5013 

Application: KiCad

Version: (5.99.0-2499-g6e4d4401a-dirty), debug build

Libraries:
	wxWidgets 3.0.4
	libcurl/7.69.1 OpenSSL/1.1.1g-fips zlib/1.2.11 brotli/1.0.7 libidn2/2.3.0 libpsl/0.21.0 (+libidn2/2.3.0) libssh/0.9.4/openssl/zlib nghttp2/1.41.0

Platform: Linux 5.7.10-201.fc32.x86_64 x86_64, 64 bit, Little endian, wxGTK

Build Info:
	Date: Jul 31 2020 16:00:57
	wxWidgets: 3.0.4 (wchar_t,wx containers,compatible with 2.8) GTK+ 3.24
	Boost: 1.69.0
	OCC: 7.4.0
	Curl: 7.69.1
	ngspice: 32
	Compiler: GCC 10.2.1 with C++ ABI 1014

Build settings:
	KICAD_SCRIPTING=ON
	KICAD_SCRIPTING_MODULES=ON
	KICAD_SCRIPTING_PYTHON3=ON
	KICAD_SCRIPTING_WXPYTHON=ON
	KICAD_SCRIPTING_WXPYTHON_PHOENIX=ON
	KICAD_SCRIPTING_ACTION_MENU=ON
	BUILD_GITHUB_PLUGIN=ON
	KICAD_USE_OCC=ON
	KICAD_SPICE=ON
	KICAD_STDLIB_DEBUG=OFF
	KICAD_STDLIB_LIGHT_DEBUG=OFF
	KICAD_SANITIZE=OFF

    Designs

      Child items ...

      2. Activity

      • Fabien Corona
        Fabien Corona @Drinausaur
        Maintainer

        Same here, on a nightly, but when I selected the color panel, not just the preference menu. Note that I deleted my .config/kicadnightly/5.99 folder beforehand (so I should have the default settings ?) I attached a video

        colorcrash-2020-08-01_01.29.06

        Application: Eeschema

Version: 5.99.0-unknown-87ebd34~102~ubuntu20.04.1, release build

Libraries:
	wxWidgets 3.0.4
	libcurl/7.68.0 OpenSSL/1.1.1f zlib/1.2.11 brotli/1.0.7 libidn2/2.2.0 libpsl/0.21.0 (+libidn2/2.2.0) libssh/0.9.3/openssl/zlib nghttp2/1.40.0 librtmp/2.3

Platform: Linux 5.4.0-42-generic x86_64, 64 bit, Little endian, wxGTK

Build Info:
	Date: Jul 31 2020 10:17:42
	wxWidgets: 3.0.4 (wchar_t,wx containers,compatible with 2.8) GTK+ 3.24
	Boost: 1.71.0
	OCE: 6.9.1
	Curl: 7.68.0
	ngspice: 31
	Compiler: GCC 9.3.0 with C++ ABI 1013

Build settings:
	KICAD_SCRIPTING=ON
	KICAD_SCRIPTING_MODULES=ON
	KICAD_SCRIPTING_PYTHON3=ON
	KICAD_SCRIPTING_WXPYTHON=ON
	KICAD_SCRIPTING_WXPYTHON_PHOENIX=ON
	KICAD_SCRIPTING_ACTION_MENU=ON
	BUILD_GITHUB_PLUGIN=ON
	KICAD_USE_OCE=ON
	KICAD_SPICE=ON
      • Jeff Young added prioritycritical statusconfirmed labels and removed priorityundecided statusnew labels

        added prioritycritical statusconfirmed labels and removed priorityundecided statusnew labels

      • Jeff Young
        Jeff Young @jeffyoung
        Maintainer

        (Note: confirmed is based on above-two reports. I have not verified it myself.)

      • Jeff Young changed milestone to %6.0.0-rc1

        changed milestone to %6.0.0-rc1

      • Ian McInerney
        Ian McInerney @imcinerney
        Maintainer

        Here is a better stacktrace:

        =================================================================
==9667==ERROR: AddressSanitizer: SEGV on unknown address 0x000000000150 (pc 0x7fdc6a05445a bp 0x7ffdded09e10 sp 0x7ffdded09dd0 T0)
==9667==The signal is caused by a READ memory access.
==9667==Hint: address points to the zero page.
    #0 0x7fdc6a05445a in std::_Hashtable<int, std::pair<int const, KIGFX::COLOR4D>, std::allocator<std::pair<int const, KIGFX::COLOR4D> >, std::__detail::_Select1st, std::equal_to<int>, std::hash<int>, std::__detail::_Mod_range_hashing, std::__detail::_Default_ranged_hash, std::__detail::_Prime_rehash_policy, std::__detail::_Hashtable_traits<false, false, true> >::_M_bucket_index(int const&, unsigned long) const /usr/bin/../lib/gcc/x86_64-redhat-linux/10/../../../../include/c++/10/bits/hashtable.h:682:60
    #1 0x7fdc6a055f0b in std::_Hashtable<int, std::pair<int const, KIGFX::COLOR4D>, std::allocator<std::pair<int const, KIGFX::COLOR4D> >, std::__detail::_Select1st, std::equal_to<int>, std::hash<int>, std::__detail::_Mod_range_hashing, std::__detail::_Default_ranged_hash, std::__detail::_Prime_rehash_policy, std::__detail::_Hashtable_traits<false, false, true> >::count(int const&) const /usr/bin/../lib/gcc/x86_64-redhat-linux/10/../../../../include/c++/10/bits/hashtable.h:1485:27
    #2 0x7fdc6a04fc1c in std::unordered_map<int, KIGFX::COLOR4D, std::hash<int>, std::equal_to<int>, std::allocator<std::pair<int const, KIGFX::COLOR4D> > >::count(int const&) const /usr/bin/../lib/gcc/x86_64-redhat-linux/10/../../../../include/c++/10/bits/unordered_map.h:938:21
    #3 0x7fdc6a04d9c8 in COLOR_SETTINGS::GetDefaultColor(int) master/build/debug/../../common/settings/color_settings.cpp:332:26
    #4 0x7fdc69ab2148 in PANEL_COLOR_SETTINGS::createSwatch(int, wxString const&) master/build/debug/../../common/dialogs/panel_color_settings.cpp:204:49
    #5 0x7fdc68e901c8 in PANEL_EESCHEMA_COLOR_SETTINGS::createSwatches() master/build/debug/../../eeschema/dialogs/panel_eeschema_color_settings.cpp:205:9
    #6 0x7fdc68e8f3ab in PANEL_EESCHEMA_COLOR_SETTINGS::PANEL_EESCHEMA_COLOR_SETTINGS(SCH_BASE_FRAME*, wxWindow*) master/build/debug/../../eeschema/dialogs/panel_eeschema_color_settings.cpp:80:5
    #7 0x7fdc691a43ab in SCH_EDIT_FRAME::InstallPreferences(PAGED_DIALOG*, PANEL_HOTKEYS_EDITOR*) master/build/debug/../../eeschema/eeschema_config.cpp:227:27
    #8 0x7fdc69ccc0fa in EDA_BASE_FRAME::OnPreferences(wxCommandEvent&) master/build/debug/../../common/eda_base_frame.cpp:668:20
    #9 0x7fdc808254cd in wxEvtHandler::ProcessEventIfMatchesId(wxEventTableEntryBase const&, wxEvtHandler*, wxEvent&) (/lib64/libwx_baseu-3.0.so.0+0x1f24cd)
    #10 0x7fdc808255ca in wxEventHashTable::HandleEvent(wxEvent&, wxEvtHandler*) (/lib64/libwx_baseu-3.0.so.0+0x1f25ca)
    #11 0x7fdc8082591f in wxEvtHandler::TryHereOnly(wxEvent&) (/lib64/libwx_baseu-3.0.so.0+0x1f291f)
    #12 0x7fdc69ccf26b in EDA_BASE_FRAME::ProcessEvent(wxEvent&) master/build/debug/../../common/eda_base_frame.cpp:219:19
    #13 0x7fdc80825782 in wxEvtHandler::DoTryChain(wxEvent&) (/lib64/libwx_baseu-3.0.so.0+0x1f2782)
    #14 0x7fdc80825a80 in wxEvtHandler::ProcessEvent(wxEvent&) (/lib64/libwx_baseu-3.0.so.0+0x1f2a80)
    #15 0x7fdc80daf9a9 in wxWindowBase::TryAfter(wxEvent&) (/lib64/libwx_gtk3u_core-3.0.so.0+0x4859a9)
    #16 0x7fdc808257da in wxEvtHandler::SafelyProcessEvent(wxEvent&) (/lib64/libwx_baseu-3.0.so.0+0x1f27da)
    #17 0x7fdc80d5df2d in wxMenuBase::SendEvent(int, int) (/lib64/libwx_gtk3u_core-3.0.so.0+0x433f2d)
    #18 0x7fdc80c51902  (/lib64/libwx_gtk3u_core-3.0.so.0+0x327902)
    #19 0x7fdc7ed0a889 in g_closure_invoke (/lib64/libgobject-2.0.so.0+0x13889)
    #20 0x7fdc7ed1d422  (/lib64/libgobject-2.0.so.0+0x26422)
    #21 0x7fdc7ed23af8 in g_signal_emit_valist (/lib64/libgobject-2.0.so.0+0x2caf8)
    #22 0x7fdc7ed23c62 in g_signal_emit (/lib64/libgobject-2.0.so.0+0x2cc62)
    #23 0x7fdc7f20669b in gtk_widget_activate /usr/src/debug/gtk3-3.24.20-1.fc32.x86_64/gtk/gtkwidget.c:7841:7
    #24 0x7fdc7f0c87dd in gtk_menu_shell_activate_item /usr/src/debug/gtk3-3.24.20-1.fc32.x86_64/gtk/gtkmenushell.c:1375:3
    #25 0x7fdc7f0c8ab2 in gtk_menu_shell_button_release /usr/src/debug/gtk3-3.24.20-1.fc32.x86_64/gtk/gtkmenushell.c:791:19
    #26 0x7fdc7f25aeab in _gtk_marshal_BOOLEAN__BOXEDv /usr/src/debug/gtk3-3.24.20-1.fc32.x86_64/gtk/gtkmarshalers.c:129:14
    #27 0x7fdc7ed0aae5  (/lib64/libgobject-2.0.so.0+0x13ae5)
    #28 0x7fdc7ed22e4a in g_signal_emit_valist (/lib64/libgobject-2.0.so.0+0x2be4a)
    #29 0x7fdc7ed23c62 in g_signal_emit (/lib64/libgobject-2.0.so.0+0x2cc62)
    #30 0x7fdc7f203fe3 in gtk_widget_event_internal /usr/src/debug/gtk3-3.24.20-1.fc32.x86_64/gtk/gtkwidget.c:7808:4
    #31 0x7fdc7f0b2c6f in propagate_event_up /usr/src/debug/gtk3-3.24.20-1.fc32.x86_64/gtk/gtkmain.c:2587:25
    #32 0x7fdc7f0b2c6f in propagate_event /usr/src/debug/gtk3-3.24.20-1.fc32.x86_64/gtk/gtkmain.c:2690:5
    #33 0x7fdc7f0b4952 in gtk_main_do_event /usr/src/debug/gtk3-3.24.20-1.fc32.x86_64/gtk/gtkmain.c:1920:9
    #34 0x7fdc7f0b4952 in gtk_main_do_event /usr/src/debug/gtk3-3.24.20-1.fc32.x86_64/gtk/gtkmain.c:1690:1
    #35 0x7fdc7ed8d028 in _gdk_event_emit gdkevents.c:73:6
    #36 0x7fdc7edc18e5 in gdk_event_source_dispatch gdkeventsource.c:367:7
    #37 0x7fdc7ea5b7ae in g_main_context_dispatch (/lib64/libglib-2.0.so.0+0x527ae)
    #38 0x7fdc7ea5bb37  (/lib64/libglib-2.0.so.0+0x52b37)
    #39 0x7fdc7ea5be52 in g_main_loop_run (/lib64/libglib-2.0.so.0+0x52e52)
    #40 0x7fdc7f0b391c in gtk_main /usr/src/debug/gtk3-3.24.20-1.fc32.x86_64/gtk/gtkmain.c:1328:7
    #41 0x7fdc80bcdda4 in wxGUIEventLoop::DoRun() (/lib64/libwx_gtk3u_core-3.0.so.0+0x2a3da4)
    #42 0x7fdc806fd630 in wxEventLoopBase::Run() (/lib64/libwx_baseu-3.0.so.0+0xca630)
    #43 0x7fdc806bd428 in wxAppConsoleBase::MainLoop() (/lib64/libwx_baseu-3.0.so.0+0x8a428)
    #44 0x54fc79 in APP_SINGLE_TOP::OnRun() master/build/debug/../../common/single_top.cpp:209:26
    #45 0x7fdc80743bcb in wxEntry(int&, wchar_t**) (/lib64/libwx_baseu-3.0.so.0+0x110bcb)
    #46 0x543db3 in main master/build/debug/../../common/single_top.cpp:283:1
    #47 0x7fdc7f64c041 in __libc_start_main /usr/src/debug/glibc-2.31-17-gab029a2801/csu/../csu/libc-start.c:308:16
    #48 0x469f0d in _start (master/build/debug/eeschema/eeschema+0x469f0d)

AddressSanitizer can not provide additional info.
SUMMARY: AddressSanitizer: SEGV /usr/bin/../lib/gcc/x86_64-redhat-linux/10/../../../../include/c++/10/bits/hashtable.h:682:60 in std::_Hashtable<int, std::pair<int const, KIGFX::COLOR4D>, std::allocator<std::pair<int const, KIGFX::COLOR4D> >, std::__detail::_Select1st, std::equal_to<int>, std::hash<int>, std::__detail::_Mod_range_hashing, std::__detail::_Default_ranged_hash, std::__detail::_Prime_rehash_policy, std::__detail::_Hashtable_traits<false, false, true> >::_M_bucket_index(int const&, unsigned long) const
==9667==ABORTING
      • jean-pierre charras closed with commit 1bcc819e

        closed with commit 1bcc819e

      Please register or sign in to reply