diff options
| author |   Ingo Schwarze <schwarze@openbsd.org> | 2014-07-25 21:05:41 +0000 |
|---|---|---|
| committer | Ingo Schwarze <schwarze@openbsd.org> | 2014-07-25 21:05:41 +0000 |
| commit | 8b17a548dd118d115762ef272ab9eb59b2f5119b (patch) | |
| tree | 27c8ce06fa2d9c9d8c41b4c4393397ca897af53a /compat_ohash.c | |
| parent | f5d413e221cddb98e1425f70917934bb8cca9514 (diff) | |
| download | mandoc-8b17a548dd118d115762ef272ab9eb59b2f5119b.tar.gz mandoc-8b17a548dd118d115762ef272ab9eb59b2f5119b.tar.zst mandoc-8b17a548dd118d115762ef272ab9eb59b2f5119b.zip | |
Choosing the right encoding is a tricky business...
Printing query strings for URIs *always* needs URI-encoding, and when
embedding the URI into an HTML document, it needs replacement of
the "&" separators by "&" *in addition to that*, not instead.
Delete the function html_primtquery(), it was completely wrong.
You can see the badness by entering "mandoc &sec=2" into the query input
box before this patch and click "Submit". You come to the right page at
first (...man.cgi?query=mandoc+%26sec%3D2&apropos=0&sec=0&...), but now
the link to mandoc(1) is wrong: ...mandoc.1?query=mandoc &sec=2&...
Clicking on that, the "&sec=2" disappears from the query input box and
suddenly you have the first dropdown set to "2 - System Calls". Oops.
Diffstat (limited to 'compat_ohash.c')
0 files changed, 0 insertions, 0 deletions