Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
[connectionagent] Set delayedTethering to false when clearing tetheri…
…ngWifiTech pointer. Fixes JB#38937

The assumption seems to be that if delayedTethering is true then
tetheringWifiTech is never NULL. If that happens not to be the case
then QConnectionAgent::serviceStateChanged can (and does) crash.
  • Loading branch information
monich committed Jun 12, 2017
1 parent 3ba10b0 commit 944086b
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions connd/qconnectionagent.cpp
@@ -1,9 +1,8 @@
/****************************************************************************
**
** Copyright (C) 2014 Jolla Ltd
** Copyright (C) 2014-2017 Jolla Ltd
** Contact: lorn.potter@gmail.com
**
**
** GNU Lesser General Public License Usage
** This file may be used under the terms of the GNU Lesser
** General Public License version 2.1 as published by the Free Software
Expand Down Expand Up @@ -484,6 +483,7 @@ void QConnectionAgent::techChanged()
knownTechnologies.clear();
}
if (!netman->getTechnology("wifi")) {
delayedTethering = false;
tetheringWifiTech = 0;
return;
}
Expand Down

0 comments on commit 944086b

Please sign in to comment.