[Tickets #724] NEW: smbpasswd needs to close pipe

[bugs at bugs.horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/?id=724
-----------------------------------------------------------------------
 Ticket     | 724
 Created By | ctnpublic-horde at yahoo.com
 Summary    | smbpasswd needs to close pipe
 Queue      | Passwd
 Version    | HEAD
 State      | Unconfirmed
 Priority   | 1. Low
 Type       | Bug
 Owners     | 
-----------------------------------------------------------------------


ctnpublic-horde at yahoo.com (2004-10-26 00:01) wrote:

My smbpasswd.php does not get any output unless the command exec pipe is
properly closed.

cvs diff -u -r HEAD passwd/lib/Driver/smbpasswd.php
Index: passwd/lib/Driver/smbpasswd.php
===================================================================
RCS file: /repository/passwd/lib/Driver/smbpasswd.php,v
retrieving revision 1.18
diff -u -r1.18 smbpasswd.php
--- passwd/lib/Driver/smbpasswd.php     6 Apr 2004 00:11:33 -0000      
1.18
+++ passwd/lib/Driver/smbpasswd.php     26 Oct 2004 06:59:01 -0000
@@ -110,6 +110,9 @@
             return $res;
         }

+       // must close the pipe else we'll get no output
+       pclose($this->_fp);
+
         $res = file($tmpfile);
         if (strstr($res[count($res) - 1], 'Password changed for user') ===
false) {
             return PEAR::raiseError(strrchr(trim($res[count($res) - 2]),
':'));