From e9152c877218613c132c982bee840a7fdf871162 Mon Sep 17 00:00:00 2001 From: Pete Deas Date: Fri, 29 Apr 2016 21:16:13 +0100 Subject: [PATCH] gopher: ensure slash after entity type in links This prevents links with null selectors being interpreted as pointing to a (likely non-existent) file gopher://server/1 An example of a page with such links: gopher://gopher.floodgap.com/1/new --- src/protocol/gopher/gopher.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/src/protocol/gopher/gopher.c b/src/protocol/gopher/gopher.c index dcca0a66..da76d37f 100644 --- a/src/protocol/gopher/gopher.c +++ b/src/protocol/gopher/gopher.c @@ -527,6 +527,10 @@ add_gopher_menu_line(struct string *buffer, unsigned char *line) add_format_to_string(&address, "gopher://%s/%c", host, entity); + /* Ensure we put a slash after entity type */ + if (selector[0] != '/') + add_char_to_string(&address, '/'); + /* Encode selector string */ encode_selector_string(&address, selector); }