summaryrefslogtreecommitdiffstats
path: root/src
diff options
context:
space:
mode:
authorFrerich Raabe <raabe@froglogic.com>2016-03-02 16:40:57 +0100
committerKatja Marttila <katja.marttila@theqtcompany.com>2016-03-04 07:16:04 +0000
commit408bddb4d84b7720353e0cd1d0eaf9062f3f65f1 (patch)
treedc722b54112e8c1d7a93c00d6012fa0458750838 /src
parentb09fa7168314fd5ed0389eeb264c8dfdd2f97a61 (diff)
Fixed file handle leak when sudo'ing
The write handle of the stderr pipe was closed in the parent process further up already. What's left to close is the read handle. Change-Id: I988c5334edb2df002cb29b1f2d47fb34eb284185 Reviewed-by: Katja Marttila <katja.marttila@theqtcompany.com>
Diffstat (limited to 'src')
-rw-r--r--src/libs/installer/adminauthorization_x11.cpp2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/libs/installer/adminauthorization_x11.cpp b/src/libs/installer/adminauthorization_x11.cpp
index cab62a32f..3f8702eac 100644
--- a/src/libs/installer/adminauthorization_x11.cpp
+++ b/src/libs/installer/adminauthorization_x11.cpp
@@ -225,7 +225,7 @@ bool AdminAuthorization::execute(QWidget *parent, const QString &program, const
printError(parent, QString::fromLocal8Bit(errData.constData()));
}
- ::close(pipedData[1]);
+ ::close(pipedData[0]);
return success;
}