[issue56] Crash during RPM Installation

[Rob Ludwick at Labix Tracker]

Rob Ludwick <rob at rcludw.no-ip.org> added the comment:

Wild guess here.  Is it possible in Protectedread() that self.bytebuffer doesn't
exist yet?  

self.charbuffer looks protected against this by the first line in Protectedread():

    if not hasattr(self, "charbuffer"):
        self.charbuffer = u""

self.bytebuffer is also set down around line 48:

   self.bytebuffer = data[decodedbytes:]
    
The error occurs on line 38:
   
   data = self.bytebuffer + newdata

A cursory search shows that bytebuffer is only referenced by Protectedread.  So
it doesn't look like it's referenced anywhere else.  

So I'm thinking that you could do the same hasattr() magic on bytebuffer.
  
    if hasattr(self, bytebuffer):
         self.bytebuffer = ""

_______________________________________
Labix issue tracker <tracker at labix.org>
<http://tracker.labix.org/issue56>
_______________________________________