KiCad PCB EDA Suite
Loading...
Searching...
No Matches
test_view_null_safety.cpp
Go to the documentation of this file.
1/*
2 * This program source code file is part of KiCad, a free EDA CAD application.
3 *
4 * Copyright The KiCad Developers, see AUTHORS.txt for contributors.
5 *
6 * This program is free software; you can redistribute it and/or
7 * modify it under the terms of the GNU General Public License
8 * as published by the Free Software Foundation; either version 2
9 * of the License, or (at your option) any later version.
10 *
11 * This program is distributed in the hope that it will be useful,
12 * but WITHOUT ANY WARRANTY; without even the implied warranty of
13 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
14 * GNU General Public License for more details.
15 *
16 * You should have received a copy of the GNU General Public License
17 * along with this program; if not, you may find one here:
18 * http://www.gnu.org/licenses/old-licenses/gpl-2.0.html
19 * or you may search the http://www.gnu.org website for the version 2 license,
20 * or you may write to the Free Software Foundation, Inc.,
21 * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA
22 */
23
31
33
34#include <view/view.h>
35#include <view/view_group.h>
36
37using namespace KIGFX;
38
39
40BOOST_AUTO_TEST_SUITE( ViewNullSafety )
41
42
43BOOST_AUTO_TEST_CASE( HideAndSetVisibleTolerateNullItem )
44{
45 VIEW view;
46
47 // Before the fix each of these dereferenced the null item via viewPrivData() and aborted.
48 view.Hide( nullptr, true );
49 view.Hide( nullptr, false, true );
50 view.SetVisible( nullptr, true );
51 view.SetVisible( nullptr, false );
52
53 // Reaching this point without crashing is the assertion.
54 BOOST_TEST( true );
55}
56
57
58BOOST_AUTO_TEST_CASE( ViewGroupRejectsNullItem )
59{
60 // Issue #24778: a re-entrant ExitGroup() during SCH_SELECTION_TOOL::EnterGroup() left the
61 // group overlay with a null (stale) member. VIEW_GROUP stored it, then the next repaint
62 // dereferenced it in ViewBBox()/ViewDraw(). Add() must drop a null the same way the VIEW
63 // mutators do.
65
66 group.Add( nullptr );
67
68 BOOST_TEST( group.GetSize() == 0u );
69
70 // Without the guard this iterated m_groupItems[0] on a null item and crashed.
71 group.ViewBBox();
72}
73
74
Extend VIEW_ITEM by possibility of grouping items into a single object.
Definition view_group.h:39
Hold a (potentially large) number of VIEW_ITEMs and renders them on a graphics device provided by the...
Definition view.h:63
void Hide(VIEW_ITEM *aItem, bool aHide=true, bool aHideOverlay=false)
Temporarily hide the item in the view (e.g.
Definition view.cpp:1780
void SetVisible(VIEW_ITEM *aItem, bool aIsVisible=true)
Set the item visibility.
Definition view.cpp:1756
The Cairo implementation of the graphics abstraction layer.
Definition eda_group.h:29
BOOST_AUTO_TEST_CASE(HorizontalAlignment)
BOOST_AUTO_TEST_SUITE(CadstarPartParser)
BOOST_AUTO_TEST_SUITE_END()
BOOST_TEST(netlist.find("R_G1 ARM_OUT1 DIE_B R='0.001 / ((SW_STATE)") !=std::string::npos)
BOOST_AUTO_TEST_CASE(HideAndSetVisibleTolerateNullItem)