-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 severity 301607 critical thanks
Hi again, I just discovered that double quotes in long descripions get stripped too, when the auto-update feature is enabled. That specific one is solved with the attached patch. But what worries me most, though, is that the complete control.in file is interpreted as a shell script. What if someone (in good faith) had a small code example included in a long description like this: Don't use this package to do "my_package > /etc/passwd" directly. That wouldn't just break - it would do funny things to your build environment!!! Thank goodness we all always use fakeroot - right? I dare raise the severity of this bug. Yes, I am aware that DEB_AUTO_UPDATE_DEBIAN_CONTROL is an experimental feature, but still it shouldn't be able to cause surprises as fatal as these. I very strongly suggest not to trust the complete debian/control file to be interpreted sanely as a shell script. Let's figure out a way to only mess with explicitly selected lines of the file instead. - Jonas - -- * Jonas Smedegaard - idealist og Internet-arkitekt * Tlf.: +45 40843136 Website: http://dr.jones.dk/ - Enden er nÃr: http://www.shibumi.org/eoti.htm -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.0 (GNU/Linux) Comment: Using GnuPG with Thunderbird - http://enigmail.mozdev.org iD8DBQFCT6j7n7DbMsAkQLgRAvUjAJwI1q/Ew6uoLAwCIFc0IA4PI6foGACgilia iTyfEDwktlTnNtA/1aytB+c= =IJiJ -----END PGP SIGNATURE-----
--- /usr/share/cdbs/1/rules/buildcore.mk 2005-03-10 13:35:06.000000000 +0100 +++ buildcore.mk 2005-04-03 10:07:28.000000000 +0200 @@ -211,6 +211,7 @@ -e "s/\[cpu: \([^]]*\)\]/\[\`type-handling \\\\\`echo \1 | tr ' ' ','\\\\\` any\`\]/g" \ -e "s/\[system: \([^]]*\)\]/\[\`type-handling any \\\\\`echo \1 | tr ' ' ','\\\\\`\`\]/g" \ \ + -e "s/\"/\\\\\"/g" \ -e "s/^/echo \"/g" \ -e "s/\\$$/\\\\$$/g" \ -e "s/$$/\"/g" \