GitHub PushEvent at 2014-05-01 09:03:12
Repository: osTicket-1.8
Actor: NoobPsyBot
Payload: {"shas":[["34d8d0b405928fb2401b9a3bfbf86f0a401069dd","420d97235124da5bf24c51a35edb1119f653ce09@osticket.com","tnef: Fix major issue iterating over attr streams\n\nThe original logic would read the count of attributes in the stream and then\nread the first attribute in the constructor of TnefAttributeStreamReader.\nThen the iterator interface would call ::rewind() before iterating to the\nfirst item. rewind() set the @pos attribute to zero, which would cause the\nattribute count (4-byte int) to be interpreted incorrectly as part of the\nfirst attribute.\n\nThe new logic sets the position at 4 after rewind()ing, and does not read\nthe first attribute twice. It also properly detects the end of the attribute\nstream by the number of attributes advertised as the first four bytes of the\nstream (read into the @count attribute initially).","Jared Hancock",true],["d4fed2cd02ad52bf48927c8590a2aa032aa00214","4b8373d016f277527198385ba72fda0feb5da015@rotich.com","Merge pull request #825 from greezybacon/issue/tnef-bad-rewind\n\ntnef: Fix major issue iterating over attar streams\r\n\r\nReviewed-By: Peter Rotich <peter@osticket.com>","Peter Rotich",true]],"size":2,"ref":"refs/heads/develop","head":"d4fed2cd02ad52bf48927c8590a2aa032aa00214"}