Skip to content

vendor : update cpp-httplib to 0.43.4#22888

Merged
angt merged 1 commit intoggml-org:masterfrom
cabelo:cpp-httplib-0.43.4
May 10, 2026
Merged

vendor : update cpp-httplib to 0.43.4#22888
angt merged 1 commit intoggml-org:masterfrom
cabelo:cpp-httplib-0.43.4

Conversation

@cabelo
Copy link
Copy Markdown
Contributor

@cabelo cabelo commented May 9, 2026

Overview

Additional information

Requirements

What's Changed

v0.43.4

Security / bug fixes

  • Reject malformed chunk-size in chunked decoder: strtoul silently accepted a leading - and wrapped via
    unsigned arithmetic, so chunk-size -2 produced ULONG_MAX-1, bypassing the ULONG_MAX guard and letting
    a client drive the server toward unbounded allocation. Replaced with a manual hex parser that requires at
    least one hex digit, detects size_t overflow per digit, and accepts only chunk-ext or end-of-line after
    the digits (RFC 9112 §7.1)
  • Fix only invoke setarch on Linux in test/Makefile so the test build works on FreeBSD and other
    non-Linux systems where setarch is unavailable

@cabelo cabelo requested a review from ggerganov as a code owner May 9, 2026 22:37
@github-actions github-actions Bot added script Script related python python script changes labels May 9, 2026
@cabelo
Copy link
Copy Markdown
Contributor Author

cabelo commented May 9, 2026

@angt Sorry, but this PR is a fix, that's why I'm bothering you again.

@taronaeo taronaeo added the merge ready A maintainer can use this label to indicate that they consider the changes final and ready to merge. label May 10, 2026
@angt angt self-requested a review May 10, 2026 16:46
@angt angt merged commit 5d5d2e1 into ggml-org:master May 10, 2026
48 of 49 checks passed
meh pushed a commit to meh/llama.cpp that referenced this pull request May 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merge ready A maintainer can use this label to indicate that they consider the changes final and ready to merge. python python script changes script Script related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants