thac0/gmnisrv
1
0
Fork 0
forked from aniani/gmnisrv
Code Pull Requests Activity

Routing: Fix non-ascii paths

Browse Source
This commit is contained in:
Zach DeCook 2021-01-20 23:46:59 -05:00 committed by Drew DeVault
parent fbef1d34a1
commit ea360fa4c1
1 changed files with 12 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
src/serve.c
View File

@ -11,6 +11,7 @@
#include <sys/types.h> #include <sys/types.h>
#include <unistd.h> #include <unistd.h>
#include "config.h" #include "config.h"
#include "escape.h"
#include "gemini.h" #include "gemini.h"
#include "log.h" #include "log.h"
#include "mime.h" #include "mime.h"
@ -377,9 +378,10 @@ serve_request(struct gmnisrv_client *client)
struct gmnisrv_route *route = host->routes; struct gmnisrv_route *route = host->routes;
assert(route); assert(route);
char *client_path = curl_unescape(client->path, 0);
char *url_path = NULL; char *url_path = NULL;
while (route) { while (route) {
if (route_match(route, client->path, &url_path)) { if (route_match(route, client_path, &url_path)) {
break; break;
} }
@ -389,6 +391,7 @@ serve_request(struct gmnisrv_client *client)
if (!route) { if (!route) {
client_submit_response(client, client_submit_response(client,
GEMINI_STATUS_NOT_FOUND, "Not found", NULL); GEMINI_STATUS_NOT_FOUND, "Not found", NULL);
free(client_path);
free(url_path); free(url_path);
return; return;
} }
@ -397,7 +400,6 @@ serve_request(struct gmnisrv_client *client)
// Paths on paths on paths on paths // Paths on paths on paths on paths
// My apologies to the stack // My apologies to the stack
char client_path[PATH_MAX + 1] = "";
char real_path[PATH_MAX + 1] = ""; char real_path[PATH_MAX + 1] = "";
char pathinfo[PATH_MAX + 1] = ""; char pathinfo[PATH_MAX + 1] = "";
char temp_path[PATH_MAX + 1] = ""; char temp_path[PATH_MAX + 1] = "";
@ -405,10 +407,10 @@ serve_request(struct gmnisrv_client *client)
if ((size_t)n >= sizeof(real_path)) { if ((size_t)n >= sizeof(real_path)) {
client_submit_response(client, GEMINI_STATUS_PERMANENT_FAILURE, client_submit_response(client, GEMINI_STATUS_PERMANENT_FAILURE,
"Request path exceeds PATH_MAX", NULL); "Request path exceeds PATH_MAX", NULL);
free(client_path);
free(url_path); free(url_path);
return; return;
} }
strcpy(client_path, client->path);
int nlinks = 0; int nlinks = 0;
struct stat st; struct stat st;
@ -455,6 +457,7 @@ serve_request(struct gmnisrv_client *client)
client_submit_response(client, client_submit_response(client,
GEMINI_STATUS_NOT_FOUND, "Not found", NULL); GEMINI_STATUS_NOT_FOUND, "Not found", NULL);
free(client_path);
free(url_path); free(url_path);
return; return;
} }
@ -462,6 +465,7 @@ serve_request(struct gmnisrv_client *client)
if (S_ISDIR(st.st_mode)) { if (S_ISDIR(st.st_mode)) {
if (route->autoindex) { if (route->autoindex) {
serve_autoindex(client, real_path); serve_autoindex(client, real_path);
free(client_path);
free(url_path); free(url_path);
return; return;
} else { } else {
@ -485,6 +489,7 @@ serve_request(struct gmnisrv_client *client)
client_submit_response(client, client_submit_response(client,
GEMINI_STATUS_NOT_FOUND, GEMINI_STATUS_NOT_FOUND,
"Not found", NULL); "Not found", NULL);
free(client_path);
free(url_path); free(url_path);
return; return;
} }
@ -503,6 +508,7 @@ serve_request(struct gmnisrv_client *client)
// Don't serve special files // Don't serve special files
client_submit_response(client, client_submit_response(client,
GEMINI_STATUS_NOT_FOUND, "Not found", NULL); GEMINI_STATUS_NOT_FOUND, "Not found", NULL);
free(client_path);
free(url_path); free(url_path);
return; return;
} }
@ -514,9 +520,12 @@ serve_request(struct gmnisrv_client *client)
serve_cgi(client, real_path, serve_cgi(client, real_path,
(const char *)client_path, (const char *)client_path,
(const char *)pathinfo); (const char *)pathinfo);
free(client_path);
return; return;
} }
free(client_path);
FILE *body = fopen(real_path, "r"); FILE *body = fopen(real_path, "r");
if (!body) { if (!body) {
if (errno == ENOENT) { if (errno == ENOENT) {