[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

please consider kaa-metadata 0.7.4-2



hi,

please consider  kaa-metadata 0.7.4-2 for Lenny.

In attachment there is
debdiff -d ~/freevo/kaa/kaa-metadata_0.7.4-1.dsc \
~/freevo/svn-alioth/packages//kaa-metadata/build-area/kaa-metadata_0.7.4-2.dsc

I added a patch (from upstream SVN) that solves a nasty bug.
The bug in brief: when incomplete AVI files are around (such as those
generated by mencoder), and when freevo would use kaa-metadata to parse
AVI headers, then it would erroneously read the whole AVI into memory,
and often crash. More details in
http://sourceforge.net/tracker2/?func=detail&aid=2140819&group_id=46652&atid=446895


The "deb" debdiff is almost empty.
$ debdiff -d ~/freevo/kaa/kaa-metadata_0.7.4-1_amd64.changes
~/freevo/svn-alioth/packages//kaa-metadata/build-area/kaa-metadata_0.7.4-2_i386.changes

File lists identical (after any substitutions)
Control files: lines which differ (wdiff format)
------------------------------------------------
Architecture: [-amd64-] {+i386+}
Installed-Size: [-884-] {+876+}
Version: [-0.7.4-1-] {+0.7.4-2+}


a.
diff -u kaa-metadata-0.7.4/debian/changelog kaa-metadata-0.7.4/debian/changelog
--- kaa-metadata-0.7.4/debian/changelog
+++ kaa-metadata-0.7.4/debian/changelog
@@ -1,3 +1,10 @@
+kaa-metadata (0.7.4-2) unstable; urgency=low
+
+  * Add a SVN patch to avoid huge memory usage on broken AVI files.
+    See http://sourceforge.net/tracker2/?func=detail&aid=2140819&group_id=46652&atid=446895
+  
+ -- A Mennucc1 <mennucc1@debian.org>  Sun, 26 Oct 2008 21:43:23 +0100
+
 kaa-metadata (0.7.4-1) unstable; urgency=low
 
   * New upstream release.
diff -u kaa-metadata-0.7.4/debian/patches/series kaa-metadata-0.7.4/debian/patches/series
--- kaa-metadata-0.7.4/debian/patches/series
+++ kaa-metadata-0.7.4/debian/patches/series
@@ -1,0 +2,2 @@
+kaa-size.diff
+
only in patch2:
unchanged:
--- kaa-metadata-0.7.4.orig/debian/patches/kaa-size.diff
+++ kaa-metadata-0.7.4/debian/patches/kaa-size.diff
@@ -0,0 +1,18 @@
+Index: metadata/src/video/riff.py
+===================================================================
+--- metadata/src/video/riff.py	(révision 3650)
++++ metadata/src/video/riff.py	(révision 3652)
+@@ -504,9 +504,12 @@
+                 self._parseLISTmovi(size-4, file)
+                 return True
+             elif size > 80000:
+-                log.debug('RIFF LIST "%s" to long to parse: %s bytes' % (key, size))
++                log.debug('RIFF LIST "%s" too long to parse: %s bytes' % (key, size))
+                 t = file.seek(size-4,1)
+                 return True
++            elif size < 5:
++                log.debug('RIFF LIST "%s" too short: %s bytes' % (key, size))
++                return True
+ 
+             t = file.read(size-4)
+             log.debug('parse RIFF LIST "%s": %d bytes' % (key, size))

Attachment: signature.asc
Description: OpenPGP digital signature


Reply to: