simple http server in C by potrue · Pull Request #81 · potrue/leetcode

potrue · 2026-02-05T13:04:40Z

No description provided.

YUTA-Uchi

一旦急ぎで見つけた部分をレビューしました。途中から自分が作った方にもたくさん不備がみつかってきたので私の方もプルリク上げたいと思います

YUTA-Uchi · 2026-02-09T10:23:01Z

+            break;
+        }
+
+        // "\r\n\r\n" (4文字) が読み込み境界を跨ぐ可能性を考慮し、3バイト戻ったところから探索する


ここから64行目までの処理は特定のOSのための処理でしょうか？
\r\nが出てくるとWindows向けの特別な処理をやっているように見えました。（\r\nがWindowsの改行文字なので）もし特別処理であればその処理だけ関数別にしたほうがわかりやすい気がします。

（正直書いた時は詳しく把握できていませんでしたが、）これは送られてきているリクエストからヘッダーを切り離す（空行を見つける）処理で、RFC9112で改行は\r\n(CRLF)で表すように決められているみたいなので、特にOS固有の処理ではなく、ヘッダーがここまでだというのを判断するためにOS共通で使用しているロジック、という感じです。
プログラムを書いた時には全然見られていませんでしたが、本当はこちらに書いてある"The normal procedure"に従ってリクエストを読み込むのが良さそうかもしれません。

YUTA-Uchi · 2026-02-09T10:25:25Z

+    }
+
+    if (strncmp(read_buffer, path_prefix, strlen(path_prefix)) != 0) {
+        respond_and_close(connection_fd, 404, "The requested URL was not found on this server\n");


しっかり追えているか不安ですが、ここでfree(read_buffer);したほうがメモリリークがないかもです

あ、これは必要そうですね！メモリリークを起こしていそうです。L97にも必要だと思います。

YUTA-Uchi · 2026-02-09T10:25:45Z

+    }
+
+    if (strncmp(read_buffer + strlen(path_prefix), query_prefix, strlen(query_prefix)) != 0) {
+        respond_and_close(connection_fd, 400, "Failed to find query");


ここも同様です。free(read_buffer);したほうがメモリリークがないかもです

YUTA-Uchi · 2026-02-09T10:28:41Z

@@ -0,0 +1,8 @@
+コンパイル・実行（Unix/Linux環境）:


https://github.com/potrue/leetcode/pull/81/changes#r2781804952
もし上記の処理がWindows向けであるならば、ここも変更が必要か検討が必要かと思います

YUTA-Uchi · 2026-02-09T10:32:02Z

+    };
+
+    if (bind(socket_fd, (struct sockaddr*)&addr, sizeof(addr)) < 0) {
+        perror("bind");


close(socket_fd);をしておいた方が良いと思います

これもあったほうが良さそうです。

YUTA-Uchi · 2026-02-09T10:32:10Z

+    }
+
+    if (listen(socket_fd, BACKLOG_SIZE) < 0) {
+        perror("listen");


close(socket_fd);をしておいた方が良いと思います

inazz · 2026-03-15T14:01:35Z

+        return;
+    }
+
+    write(connection_fd, response, response_len);


write が失敗したり、部分的にだけ書き込みが成功した場合に対応したい。

inazz · 2026-03-15T14:02:29Z

+
+static char* read_request(int connection_fd) {
+    size_t capacity = INITIAL_READ_BUFFER_SIZE;
+    char* read_buffer = malloc(capacity);


malloc が NULL を返したら?

inazz · 2026-03-15T14:12:15Z

+        return;
+    }
+
+    if (strncmp(read_buffer, path_prefix, strlen(path_prefix)) != 0) {


/calc で始まれば、他の path でも区別して動いてしまう
GET /calcium?hoge=...

inazz · 2026-03-15T14:13:51Z

+        return;
+    }
+
+    if (strncmp(read_buffer + strlen(path_prefix), query_prefix, strlen(query_prefix)) != 0) {


query 引数が第１引数だった時以外は動かない?
/calc?utm_source=github&query=1+2

inazz · 2026-03-15T14:14:23Z

+
+    char* query_start = read_buffer + strlen(path_prefix) + strlen(query_prefix);
+    int a, b;
+    if (sscanf(query_start, "%d+%d", &a, &b) != 2) {


足し算以外のケースも対応したい

inazz · 2026-03-15T14:14:53Z

@@ -0,0 +1,6 @@
+#ifndef UTILS_H


inazz · 2026-03-15T14:15:21Z

@@ -0,0 +1,8 @@
+コンパイル・実行（Unix/Linux環境）:
+
+`gcc -o server server.c utils.c`


utils.c が handler.c に変わった？

inazz · 2026-03-15T14:16:16Z

+
+動作確認:
+
+`curl http://127.0.0.1:8080/calc\?query\=1+11` など


これブラウザからだとどう入力するのが正しくなりますか? (url encoding)

inazz · 2026-03-15T14:17:27Z

+
+        int connection_fd = accept(socket_fd, (struct sockaddr*)&client_addr, &client_addr_size);
+        if (connection_fd < 0) {
+            perror("accept");


EINTR などの一時的なエラーの場合はリカバリしたい。

inazz · 2026-03-15T14:25:58Z

+        total_read_size += read_size;
+        read_buffer[total_read_size] = '\0';
+
+        if (strstr(read_buffer + str_search_offset, "\r\n\r\n")) {


データを全部読み切らずに、改行2つで早期に止めるのはどういうケースを想定をしていますか?
(keep alive?)

potrue added 2 commits February 5, 2026 22:04

simple http server in C

2993e03

update

7a9229d

YUTA-Uchi reviewed Feb 9, 2026

View reviewed changes

inazz reviewed Mar 15, 2026

View reviewed changes

		@@ -0,0 +1,8 @@
		コンパイル・実行（Unix/Linux環境）:

		@@ -0,0 +1,8 @@
		コンパイル・実行（Unix/Linux環境）:

		`gcc -o server server.c utils.c`


		動作確認:

		`curl http://127.0.0.1:8080/calc\?query\=1+11` など

Conversation

potrue commented Feb 5, 2026

Uh oh!

YUTA-Uchi left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants