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

http-c/handler.c

+            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

http-c/handler.c

+    }
+
+    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

http-c/handler.c

+    }
+
+    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

http-c/README.md

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


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

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

http-c/server.c

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


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

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

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

http-c/server.c

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


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

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

http-c/handler.c

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


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

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

http-c/handler.c

+
+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

http-c/handler.c

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


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

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

http-c/handler.c

+        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

http-c/handler.c

+
+    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

http-c/handler.h

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


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

http-c/README.md

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


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

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

http-c/README.md

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


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

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

http-c/server.c

+
+        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

http-c/handler.c

+        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