author | Lars Hjemli <hjemli@gmail.com> | 2008-10-05 10:49:46 (UTC) |
---|---|---|
committer | Lars Hjemli <hjemli@gmail.com> | 2008-10-05 10:49:46 (UTC) |
commit | a36a0d9dec8a3ba79501d2526d648e44306f0fdd (patch) (unidiff) | |
tree | ab9a6b2a0fc413887fb3fc1ddfd4fce54e26b599 | |
parent | f82b19407dd876e6c02a572615bf34b09f6fa831 (diff) | |
download | cgit-a36a0d9dec8a3ba79501d2526d648e44306f0fdd.zip cgit-a36a0d9dec8a3ba79501d2526d648e44306f0fdd.tar.gz cgit-a36a0d9dec8a3ba79501d2526d648e44306f0fdd.tar.bz2 |
html.c: add html_url_arg
This function can be used to properly escape querystring parameter values.
Signed-off-by: Lars Hjemli <hjemli@gmail.com>
-rw-r--r-- | html.c | 16 | ||||
-rw-r--r-- | html.h | 1 |
2 files changed, 17 insertions, 0 deletions
@@ -107,48 +107,64 @@ void html_ntxt(int len, char *txt) | |||
107 | html("..."); | 107 | html("..."); |
108 | } | 108 | } |
109 | 109 | ||
110 | void html_attr(char *txt) | 110 | void html_attr(char *txt) |
111 | { | 111 | { |
112 | char *t = txt; | 112 | char *t = txt; |
113 | while(t && *t){ | 113 | while(t && *t){ |
114 | int c = *t; | 114 | int c = *t; |
115 | if (c=='<' || c=='>' || c=='\'') { | 115 | if (c=='<' || c=='>' || c=='\'') { |
116 | write(htmlfd, txt, t - txt); | 116 | write(htmlfd, txt, t - txt); |
117 | if (c=='>') | 117 | if (c=='>') |
118 | html(">"); | 118 | html(">"); |
119 | else if (c=='<') | 119 | else if (c=='<') |
120 | html("<"); | 120 | html("<"); |
121 | else if (c=='\'') | 121 | else if (c=='\'') |
122 | html(""e;"); | 122 | html(""e;"); |
123 | txt = t+1; | 123 | txt = t+1; |
124 | } | 124 | } |
125 | t++; | 125 | t++; |
126 | } | 126 | } |
127 | if (t!=txt) | 127 | if (t!=txt) |
128 | html(txt); | 128 | html(txt); |
129 | } | 129 | } |
130 | 130 | ||
131 | void html_url_arg(char *txt) | ||
132 | { | ||
133 | char *t = txt; | ||
134 | while(t && *t){ | ||
135 | int c = *t; | ||
136 | if (c=='"' || c=='#' || c=='%' || c=='&' || c=='\'' || c=='+' || c=='?') { | ||
137 | write(htmlfd, txt, t - txt); | ||
138 | write(htmlfd, fmt("%%%2x", c), 3); | ||
139 | txt = t+1; | ||
140 | } | ||
141 | t++; | ||
142 | } | ||
143 | if (t!=txt) | ||
144 | html(txt); | ||
145 | } | ||
146 | |||
131 | void html_hidden(char *name, char *value) | 147 | void html_hidden(char *name, char *value) |
132 | { | 148 | { |
133 | html("<input type='hidden' name='"); | 149 | html("<input type='hidden' name='"); |
134 | html_attr(name); | 150 | html_attr(name); |
135 | html("' value='"); | 151 | html("' value='"); |
136 | html_attr(value); | 152 | html_attr(value); |
137 | html("'/>"); | 153 | html("'/>"); |
138 | } | 154 | } |
139 | 155 | ||
140 | void html_option(char *value, char *text, char *selected_value) | 156 | void html_option(char *value, char *text, char *selected_value) |
141 | { | 157 | { |
142 | html("<option value='"); | 158 | html("<option value='"); |
143 | html_attr(value); | 159 | html_attr(value); |
144 | html("'"); | 160 | html("'"); |
145 | if (selected_value && !strcmp(selected_value, value)) | 161 | if (selected_value && !strcmp(selected_value, value)) |
146 | html(" selected='selected'"); | 162 | html(" selected='selected'"); |
147 | html(">"); | 163 | html(">"); |
148 | html_txt(text); | 164 | html_txt(text); |
149 | html("</option>\n"); | 165 | html("</option>\n"); |
150 | } | 166 | } |
151 | 167 | ||
152 | void html_link_open(char *url, char *title, char *class) | 168 | void html_link_open(char *url, char *title, char *class) |
153 | { | 169 | { |
154 | html("<a href='"); | 170 | html("<a href='"); |
@@ -1,22 +1,23 @@ | |||
1 | #ifndef HTML_H | 1 | #ifndef HTML_H |
2 | #define HTML_H | 2 | #define HTML_H |
3 | 3 | ||
4 | extern int htmlfd; | 4 | extern int htmlfd; |
5 | 5 | ||
6 | extern void html_raw(const char *txt, size_t size); | 6 | extern void html_raw(const char *txt, size_t size); |
7 | extern void html(const char *txt); | 7 | extern void html(const char *txt); |
8 | extern void htmlf(const char *format,...); | 8 | extern void htmlf(const char *format,...); |
9 | extern void html_status(int code, const char *msg, int more_headers); | 9 | extern void html_status(int code, const char *msg, int more_headers); |
10 | extern void html_txt(char *txt); | 10 | extern void html_txt(char *txt); |
11 | extern void html_ntxt(int len, char *txt); | 11 | extern void html_ntxt(int len, char *txt); |
12 | extern void html_attr(char *txt); | 12 | extern void html_attr(char *txt); |
13 | extern void html_url_arg(char *txt); | ||
13 | extern void html_hidden(char *name, char *value); | 14 | extern void html_hidden(char *name, char *value); |
14 | extern void html_option(char *value, char *text, char *selected_value); | 15 | extern void html_option(char *value, char *text, char *selected_value); |
15 | extern void html_link_open(char *url, char *title, char *class); | 16 | extern void html_link_open(char *url, char *title, char *class); |
16 | extern void html_link_close(void); | 17 | extern void html_link_close(void); |
17 | extern void html_fileperm(unsigned short mode); | 18 | extern void html_fileperm(unsigned short mode); |
18 | extern int html_include(const char *filename); | 19 | extern int html_include(const char *filename); |
19 | 20 | ||
20 | extern int http_parse_querystring(char *txt, void (*fn)(const char *name, const char *value)); | 21 | extern int http_parse_querystring(char *txt, void (*fn)(const char *name, const char *value)); |
21 | 22 | ||
22 | #endif /* HTML_H */ | 23 | #endif /* HTML_H */ |