[tor-bugs] #14478 [Tor]: Zero length keys test improvements

[Tor Bug Tracker & Wiki]

#14478: Zero length keys test improvements
-----------------------------+--------------------------------
     Reporter:  cypherpunks  |      Owner:
         Type:  enhancement  |     Status:  needs_review
     Priority:  normal       |  Milestone:  Tor: 0.2.6.x-final
    Component:  Tor          |    Version:  Tor: unspecified
   Resolution:               |   Keywords:
Actual Points:               |  Parent ID:
       Points:               |
-----------------------------+--------------------------------

Comment (by cypherpunks):

 Replying to [comment:1 nickm]:
 > 2 and 3 seem like a good idea.
 >
 > I like the idea behind 1, but I'm worried about the "rm -rf".  Call me
 paranoid, but "rm -rf" is just *asking* for trouble.  Can we define a
 function to get called when the script exits, and rm the specific files
 instead?  If not we should at least make sure that the filename returned
 by mktemp is nonempty and more or less as expected, I think.

 While writing the function i ran into more possible improvements. First,
 the negative exit values seem to produce errors because negative values
 are an !''illegal number!''. Can someone verify this?
 {{{
 some at other:shell$ sh
 $ exit -1
 sh: 1: exit: Illegal number: -1
 $
 }}}
 ''Note that after the error, the shell (`sh`) does not actually exit.''

 Second, parameter expansion uses quotes to ensure it is properly expanded.
 By using `${}` the quotes become unnecessary and can be !''saved!'' for
 quoting the entire string (which should be always be done, not just in
 cases with spaces, for consistency) without the need for two double
 quotes.

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/14478#comment:3>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online