Differential D24655

give all pointers in slavebase a default init to nullptr
ClosedPublic
Actions

Authored by sitter on Oct 15 2019, 8:52 AM.

Details

Reviewers
dfaure
meven
vkrause
Commits
R241:c2bd18f17e61: give all pointers in slavebase a default init to nullptr
Summary

lest they don't get initialized and then end up crashing should get get
deleted (as is the case right now with configGroup)

Test Plan

no longer segfaulting every time rebuildConfig is run during ctest

Diff Detail

Repository
R241 KIO
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 17701
Build 17719: arc lint + arc unit
sitter created this revision.Oct 15 2019, 8:52 AM
Restricted Application added a project: Frameworks. · View Herald TranscriptOct 15 2019, 8:52 AM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
sitter requested review of this revision.Oct 15 2019, 8:52 AM
vkrause accepted this revision.Oct 15 2019, 9:06 AM
This revision is now accepted and ready to land.Oct 15 2019, 9:06 AM
Closed by commit R241:c2bd18f17e61: give all pointers in slavebase a default init to nullptr (authored by sitter). · Explain WhyOct 15 2019, 9:14 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Msrc/core/slavebase.cpp (8 lines)

Diff 67944

View Options

src/core/slavebase.cpp

Loading...