changeset:   91601:4b29d338cc41
parent:      91599:b255ecb175c4
parent:      91600:f8c9dd2626aa
user:        Victor Stinner <victor.stinner@gmail.com>
date:        Tue Jul 08 00:01:28 2014 +0200
files:       Lib/asynchat.py Lib/test/test_asynchat.py Misc/NEWS
description:
(Merge 3.4) Issue #12523: asynchat.async_chat.push() now raises a TypeError if
it doesn't get a bytes string


diff -r b255ecb175c4 -r 4b29d338cc41 Lib/asynchat.py
--- a/Lib/asynchat.py	Mon Jul 07 23:07:27 2014 +0200
+++ b/Lib/asynchat.py	Tue Jul 08 00:01:28 2014 +0200
@@ -181,6 +181,9 @@
         self.close()
 
     def push (self, data):
+        if not isinstance(data, (bytes, bytearray, memoryview)):
+            raise TypeError('data argument must be byte-ish (%r)',
+                            type(data))
         sabs = self.ac_out_buffer_size
         if len(data) > sabs:
             for i in range(0, len(data), sabs):
diff -r b255ecb175c4 -r 4b29d338cc41 Lib/test/test_asynchat.py
--- a/Lib/test/test_asynchat.py	Mon Jul 07 23:07:27 2014 +0200
+++ b/Lib/test/test_asynchat.py	Tue Jul 08 00:01:28 2014 +0200
@@ -250,6 +250,22 @@
         # (which could still result in the client not having received anything)
         self.assertGreater(len(s.buffer), 0)
 
+    def test_push(self):
+        # Issue #12523: push() should raise a TypeError if it doesn't get
+        # a bytes string
+        s, event = start_echo_server()
+        c = echo_client(b'\n', s.port)
+        data = b'bytes\n'
+        c.push(data)
+        c.push(bytearray(data))
+        c.push(memoryview(data))
+        self.assertRaises(TypeError, c.push, 10)
+        self.assertRaises(TypeError, c.push, 'unicode')
+        c.push(SERVER_QUIT)
+        asyncore.loop(use_poll=self.usepoll, count=300, timeout=.01)
+        s.join(timeout=TIMEOUT)
+        self.assertEqual(c.contents, [b'bytes', b'bytes', b'bytes'])
+
 
 class TestAsynchat_WithPoll(TestAsynchat):
     usepoll = True
diff -r b255ecb175c4 -r 4b29d338cc41 Misc/NEWS
--- a/Misc/NEWS	Mon Jul 07 23:07:27 2014 +0200
+++ b/Misc/NEWS	Tue Jul 08 00:01:28 2014 +0200
@@ -108,6 +108,9 @@
 Library
 -------
 
+- Issue #12523: asynchat.async_chat.push() now raises a TypeError if it doesn't
+  get a bytes string
+
 - Issue #21707: Add missing kwonlyargcount argument to
   ModuleFinder.replace_paths_in_code().