An interpreted, interactive, object-oriented programming language
CentOS Sources
2017-08-01 71084d584ff953f5463757ec6536406320560b4d
commit | author | age
f63228 1
CS 2 # HG changeset patch
3 # User R David Murray <rdmurray@bitdance.com>
4 # Date 1388775562 18000
5 # Node ID dd906f4ab9237020a7a275c2d361fa288e553481
6 # Parent  69b5f692455306c98aa27ecea17e6290787ebd3f
7 closes 16039: CVE-2013-1752: limit line length in imaplib readline calls.
8
9 diff --git a/Lib/imaplib.py b/Lib/imaplib.py
10 --- a/Lib/imaplib.py
11 +++ b/Lib/imaplib.py
12 @@ -35,6 +35,15 @@ IMAP4_PORT = 143
13  IMAP4_SSL_PORT = 993
14  AllowedVersions = ('IMAP4REV1', 'IMAP4')        # Most recent first
15  
16 +# Maximal line length when calling readline(). This is to prevent
17 +# reading arbitrary length lines. RFC 3501 and 2060 (IMAP 4rev1)
18 +# don't specify a line length. RFC 2683 however suggests limiting client
19 +# command lines to 1000 octets and server command lines to 8000 octets.
20 +# We have selected 10000 for some extra margin and since that is supposedly
21 +# also what UW and Panda IMAP does.
22 +_MAXLINE = 10000
23 +
24 +
25  #       Commands
26  
27  Commands = {
28 @@ -237,7 +246,10 @@ class IMAP4:
29  
30      def readline(self):
31          """Read line from remote."""
32 -        return self.file.readline()
33 +        line = self.file.readline(_MAXLINE + 1)
34 +        if len(line) > _MAXLINE:
35 +            raise self.error("got more than %d bytes" % _MAXLINE)
36 +        return line
37  
38  
39      def send(self, data):
40 diff --git a/Lib/test/test_imaplib.py b/Lib/test/test_imaplib.py
41 --- a/Lib/test/test_imaplib.py
42 +++ b/Lib/test/test_imaplib.py
43 @@ -165,6 +165,16 @@ class BaseThreadedNetworkedTests(unittes
44                                self.imap_class, *server.server_address)
45  
46  
47 +    def test_linetoolong(self):
48 +        class TooLongHandler(SimpleIMAPHandler):
49 +            def handle(self):
50 +                # Send a very long response line
51 +                self.wfile.write('* OK ' + imaplib._MAXLINE*'x' + '\r\n')
52 +
53 +        with self.reaped_server(TooLongHandler) as server:
54 +            self.assertRaises(imaplib.IMAP4.error,
55 +                              self.imap_class, *server.server_address)
56 +
57  class ThreadedNetworkedTests(BaseThreadedNetworkedTests):
58  
59      server_class = SocketServer.TCPServer