diff options
author | Frerich Raabe <raabe@froglogic.com> | 2016-03-02 16:40:57 +0100 |
---|---|---|
committer | Katja Marttila <katja.marttila@theqtcompany.com> | 2016-03-04 07:16:04 +0000 |
commit | 408bddb4d84b7720353e0cd1d0eaf9062f3f65f1 (patch) | |
tree | dc722b54112e8c1d7a93c00d6012fa0458750838 /src | |
parent | b09fa7168314fd5ed0389eeb264c8dfdd2f97a61 (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.cpp | 2 |
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; } |