Merge rev 27186 from trunk.

FileIterator.next():  the code for reading the user, description,
and extension fields from a transaction was fatally confused,
usually reading them out of the object pickle by mistake.  This
caused several tools to display binary gibberish.

branches/3.3/NEWS.txt

@@ -5,6 +5,10 @@ Release date: DD-MMM-YYYY
 Tools
 -----
 
+FileStorage.FileIterator was confused about how to read a transaction's
+user and description fields, which caused several tools to display
+binary gibberish for these values.
+
 ZODB.utils.oid_repr() changed to add a leading "0x", and to strip leading
 zeroes.  This is used, e.g., in the detail of a POSKeyError exception, to
 identify the missing oid.  Before, the output was ambiguous.  For example,

branches/3.3/src/ZODB/FileStorage/FileStorage.py

@@ -1897,16 +1897,14 @@ class FileIterator(Iterator, FileStorageFormatter):
 
             if h.status != "u":
                 pos = tpos + h.headerlen()
-                user = self._file.read(h.ulen)
-                description = self._file.read(h.dlen)
                 e = {}
                 if h.elen:
                     try:
-                        e = loads(self._file.read(h.elen))
+                        e = loads(h.ext)
                     except:
                         pass
 
-                result = RecordIterator(h.tid, h.status, user, description,
+                result = RecordIterator(h.tid, h.status, h.user, h.descr,
                                         e, pos, tend, self._file, tpos)
 
             # Read the (intentionally redundant) transaction length
@@ -1966,16 +1964,20 @@ class RecordIterator(Iterator, BaseStorage.TransactionRecord,
                     # it go to the original data like BDBFullStorage?
                     prev_txn = self.getTxnFromData(h.oid, h.back)
 
-            r = Record(h.oid, h.tid, h.version, data, prev_txn)
-
+            r = Record(h.oid, h.tid, h.version, data, prev_txn, pos)
             return r
 
         raise IndexError, index
 
 class Record(BaseStorage.DataRecord):
     """An abstract database record."""
-    def __init__(self, *args):
-        self.oid, self.tid, self.version, self.data, self.data_txn = args
+    def __init__(self, oid, tid, version, data, prev, pos):
+        self.oid = oid
+        self.tid = tid
+        self.version = version
+        self.data = data
+        self.data_txn = prev
+        self.pos = pos
 
 class UndoSearch: