Fix xp-framework/xp-framework#347

xp-framework · Jan 2, 2014 · b40d64e · b40d64e
1 parent 0021f95
commit b40d64e
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/ChangeLog.md b/ChangeLog.md
@@ -57,6 +57,11 @@ XP Framework Core ChangeLog
   - Extracted `remote` into https://github.com/xp-framework/remote
   (@kiesel, @thekid)
 
+### Bugfixes
+
+* Fixed xp-framework/xp-framework#347 - "Undefined variable: len" in BSDSocket
+  (@haimich)
+
 ### Features
 
 * Added util.ConfigurationException and util.ApplicationException as requested

diff --git a/src/main/php/peer/BSDSocket.class.php b/src/main/php/peer/BSDSocket.class.php
@@ -403,7 +403,8 @@ public function readBinary($maxLen= 4096) {
    * @throws  peer.SocketException
    */
   public function write($str) {
-    $bytesWritten= socket_write($this->_sock, $str, strlen($str));
+    $len= strlen($str);
+    $bytesWritten= socket_write($this->_sock, $str, $len);
     if (false === $bytesWritten || null === $bytesWritten) {
       $e= new SocketException('Write of '.$len.' bytes to socket failed: '.$this->getLastError());
       \xp::gc(__FILE__);