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

Please fix undefined sprintf() usage in mafft



Hello,
I'm sending you a patch for mafft. It has been reported that the usage of:

  sprintf(buf, "%s foo %d %d", buf, var1, var2);

is undefined behaviour, and should be fixed as:

  sprintf(buf + strlen(buf), " foo %d %d", var1, var");

I've made a patch that applies to the mafft package in Debian, please consider
integrating it upstream.

You can read more on the above issue at:
  <http://lists.debian.org/debian-devel/2008/12/msg01079.html>

Kindly,
David Paleino
Debian-Med Team

-- 
 . ''`.  Debian maintainer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 ----|---- http://snipr.com/qa_page
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174

Attachment: fix-undefined-sprintf-usage.patch.gz
Description: GNU Zip compressed data

Attachment: signature.asc
Description: PGP signature


Reply to: