Resolves: bug 454060
    Bug Description: ViewLog CGI crash with new adminutil (1.1.6/7)
    Reviewed by: nhosoi (Thanks!)
    Fix Description: I moved some of the url decoding code around to make it more readable.  The way it works now is that it first decodes the string in place.  This is always safe because the url encoded string (e.g. "%XX") will always take up more space than the decoded string (e.g. "x").  The next step is to scan the string looking for chars that must be replaced by the HTML entity string, and find out how much space we need to hold them.  Then, we copy the string replacing the entities.
    Platforms tested: RHEL5 x86_64
    Flag Day: no
    Doc impact: no