Package: dpkg
Version: 1.3.22

Seen in src/archives.c:

      narglist[i++]= strdup("f");
      narglist[i++]= strdup("-print0");
      narglist[i++]= 0;
      execvp(FIND, narglist);
      ohshite(_("failed to exec find for --recursive"));

The same code in dpkg 1.4.0 read:

  const char **arglist;
  ...
      arglist[i++]= "f";
      arglist[i++]= "-print0";
      arglist[i++]= 0;
      execvp(FIND, (char* const*)arglist);

and is more correct.

Whoever wrote the patch that introduced these changes was obviously
insufficiently knowledgeable about C.  All contributions by that
programmer should be reviewed by someone competent.

Ian.


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to