[htdig-dev] [patch] External parser parse bug

[Jim <li...@yg...>]

This patch corrects a command parsing bug in ExternalParser.cc. The
single argument StringList constructor only considers a tab to be a
valid separator. As such a space between a command and argument (e.g.
"convert.pl -x") is not treated as a separator, and the string is not
split properly. This results in execv calling a command that generally
doesn't exist (e.g. "convert.pl -x" rather than convert.pl).

--- htdig/orig_ExternalParser.cc	2005-05-22 23:05:18.798550988 -0600
+++ htdig/ExternalParser.cc	2005-05-22 23:05:43.890983352 -0600
@@ -227,7 +227,7 @@ ExternalParser::parse(Retriever &retriev
      int		get_file = (convertToType.length() != 0);
      String	newcontent;

-    StringList	cpargs(currentParser);
+    StringList	cpargs(currentParser, " \t");
      char   **parsargs = new char * [cpargs.Count() + 5];
      int    argi;
      for (argi = 0; argi < cpargs.Count(); argi++)