Your message dated Sat, 11 Sep 2021 17:03:42 +0000
with message-id <[email protected]>
and subject line Bug#993467: fixed in shotdetect 1.0.86-6
has caused the Debian Bug report #993467,
regarding Shotdetect built from apt source aborts with "munmap_chunk(): invalid
pointer" (Debian 10)
to be marked as done.
This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.
(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact [email protected]
immediately.)
--
993467: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=993467
Debian Bug Tracking System
Contact [email protected] with problems
--- Begin Message ---
Package: shotdetect
Version: 1.0.86-5+b2
Severity: important
File: /usr/bin/shotdetect
Dear Maintainer,
This happened on Debian10. The same code used to work fine on Debian9.
* What led up to the situation?
A: I wanted to build the application from its "apt source" package.
* What exactly did you do (or not do) that was effective (or
ineffective)?
A: I've downloaded the package source (apt source shotdetect), then
ran `debuild -us -uc`, without any modifications to the source.
* What was the outcome of this action?
A: The generated shotdetect binary exits with an "invalid pointer"
error when initializing the output folder(names).
The binary provided by the official repositories works fine though.
* What outcome did you expect instead?
I expected the created binary to work just like the one from the
repositories.
**The good news is, it's fixed :) **
In fact it's a missing "return" statement in "create_img_dir()" in
src/image.cpp.
Thanks to Georg Lippitsch and Giulio Paci for finding it :)
I've tested the fix and can provide a patch.
Here's the output of "quilt diff" after the fix:
---------------------
Index: shotdetect-1.0.86/src/image.cpp
===================================================================
--- shotdetect-1.0.86.orig/src/image.cpp
+++ shotdetect-1.0.86/src/image.cpp
@@ -66,6 +66,7 @@ image::create_img_dir ()
}
free (buf);
+ return 0;
}
int
---------------------
Thanks in advance,
Peter B.
--- End Message ---
--- Begin Message ---
Source: shotdetect
Source-Version: 1.0.86-6
Done: Giulio Paci <[email protected]>
We believe that the bug you reported is fixed in the latest version of
shotdetect, which is due to be installed in the Debian FTP archive.
A summary of the changes between this version and the previous one is
attached.
Thank you for reporting the bug, which will now be closed. If you
have further comments please address them to [email protected],
and the maintainer will reopen the bug report if appropriate.
Debian distribution maintenance software
pp.
Giulio Paci <[email protected]> (supplier of updated shotdetect package)
(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing [email protected])
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA512
Format: 1.8
Date: Sat, 11 Sep 2021 16:47:08 +0200
Source: shotdetect
Architecture: source
Version: 1.0.86-6
Distribution: unstable
Urgency: medium
Maintainer: Debian Multimedia Maintainers <[email protected]>
Changed-By: Giulio Paci <[email protected]>
Closes: 963965 993467
Changes:
shotdetect (1.0.86-6) unstable; urgency=medium
.
[ Ondřej Nový ]
* d/copyright: Use https protocol in Format field.
* d/control: Set Vcs-* to salsa.debian.org.
.
[ Felipe Sateler ]
* Change maintainer address to [email protected].
.
[ Giulio Paci ]
* d/rules: Migrate from CDBS to dh.
* d/rules: Backup and restore some files during building and clean targets.
* d/control: Update Homepage (Closes: #963965).
* d/control: Set Rules-Requires-Root to no.
* d/control: Add pkg-config build dependency.
* d/control: Set debhelper-compat to 13.
* d/control: Drop unnecessary dependencies.
* d/copyright: Update Source and Upstream-Contact.
* d/copyright: Remove file sections related to debian/* and autotools.
* Add 1009-fix_missing_return_statement.patch (Closes: #993467).
* Add 1010-update_autotools_files.patch.
* Remove trailing space from 1008-fix-FTBFS-ffmpeg_2.9.patch.
* Add header to libav10.patch.
* Drop unused 1005-add_framenumber_to_filename.patch and
1006-label_xaxis_with_timecode.patch.
* Drop d/control.in, d/copyright_hints, d/compat.
* Bump Standards-Version to 4.6.0.
No further changes needed.
Checksums-Sha1:
f0b846aad249260b2b29912e78d5ed3afad19948 2136 shotdetect_1.0.86-6.dsc
3b5f8f649eeac87c562216b3c3136c348551c1b3 10416
shotdetect_1.0.86-6.debian.tar.xz
Checksums-Sha256:
62e8546d4c12acf0ac8485124a8c5212b76b55eea23e4c26d82f4f61475020ba 2136
shotdetect_1.0.86-6.dsc
fae5f3374bfe836faf7007060ae73fcc817f65ba76fce1807f74d4933d5f0a0b 10416
shotdetect_1.0.86-6.debian.tar.xz
Files:
9a5a72c8ebcd77c35655b9bb11244428 2136 video optional shotdetect_1.0.86-6.dsc
f7bcf73c39c1791a9f20669bddfec90c 10416 video optional
shotdetect_1.0.86-6.debian.tar.xz
-----BEGIN PGP SIGNATURE-----
iQIzBAEBCgAdFiEE94y6B4F7sUmhHTOQafL8UW6nGZMFAmE83VkACgkQafL8UW6n
GZOFHBAApTaljaFLUafroyWnuw4NkS6aoaoDyKCSLuN2FYP84z7sdtxhQMt+lg/X
6oheHB72NKhIoQo/fBKtewuVsCk41lg3jLrJZOd+YMYGKVbBTULtny/Fb8J6/yDe
IBvJiOLQpK18ShoAi3NlQtBNM/fjSqC3YlqwTajY+xeJqq0tu930H4PzDPLftypT
rJ/yPD7ugNUZ9Ec3yqidyCQxfsc9gP6Ts/B75hszJDMzVI3uRN5NUZnWUAvioVEw
5DgX6wjkd4GtoIMjwLi56LOnmsUor9eKISkE2IkUShWNItGXW4wXr0HWcxg2o3q+
Yd4dx6/OAK9LOxDoTyIfDSNEvSVOPJZsWdDavzBLqjWr/8CfrXrOffhDAzsbclDo
I9GiEaPcbxSPz0UPObgRf3xdbQEcO62NEhr885T99Tn2NkZGHx5QwqAlBm8ycdiM
QpR8QwHKIvf+5GI67Gbgez6635wz7U9DWpmoU9Z/xoePa1oGl14lALXF+lyRGKFi
GYE/36xJ8RvRhpTL6fwMRQnDwu9D7md1DmkGUy0Lk7mB5qa56AzfS4RC2AnKb4zp
gf5eilLV5a2rV4iuf+2G8b/jhj6XJtPL0Mp3cI12U3ENkygFdvB/Gf9lRuXjJ1VA
RkLgG8o5dYucA3RzKljuP5vJV/OkNojgDhrXuZq7DbAldk1aFT4=
=NPcl
-----END PGP SIGNATURE-----
--- End Message ---
