|
|||||||||||
|
linux-ipsec: more problems
From: Henry Spencer <henry(at)zoo.utoronto.ca>
Date: Wed Mar 04 1998 - 15:48:51 EST
Along with --version, the commands need to recognize --help, so one can get a syntax summary without having to fumble around trying to provoke a syntax error. The program that dumps core when invoked without arguments was probably tncfg. It is absolutely necessary for operational use that there be a way for configuration commands to fetch things like keys from files, doing it *themselves* rather than using shell syntax to make the shell do it. The problem with the shell is that it can supply such data to the programs only as arguments or environment variables, and both of those routes are exposed to scrutiny by "ps". The programs must do the file reads themselves. To maximize portability, source code probably ought to avoid using gcc compiler functionality that goes beyond standard C -- for example, inline functions -- although this is not an immediate problem. CVS/RCS comments that lie are very bad; they should be truthful or should not be present at all. The Ottawa Pluto interoperability bug may justify a right-way/wrong-way flag even if we were in the right. (We haven't gotten any useful testing done in that area yet, but problems in that area do seem possible.)
"usage error in gparse" is not a good syntax-error report -- it is highly
We need to invest some effort defining good standard notations for things like SAs, and then fixing *everything* to use them. In particular, input and output should use the same notations; there is far too much cut-and- paste currently needed when constructing input based on output. Speaking of standard notation: it should never be necessary to remember which inputs/outputs are hexadecimal, because hex numbers should always begin with "0x", on both input and output. This notation may not be ideal but it is the accepted standard. The /proc file which shows the result of an eroute command has a name ending in "route", not "eroute" -- such inconsistencies can be confusing and should be fixed.
Invocations of perror() should be systematically rooted out of the code,
because they almost never give enough information to be useful. Even
Henry Spencer
henry@zoo.toronto.edu
Received on Wed Mar 4 16:47:14 1998This archive was generated by hypermail 2.1.8 : Wed Aug 23 2006 - 12:59:27 EDT |
||||||||||
|
|||||||||||