Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

fix: some bugs in DNS proxy#201

Merged
aarond10 merged 1 commit into
aarond10:masteraarond10/https_dns_proxy:masterfrom
toxeh:mastertoxeh/https_dns_proxy:masterCopy head branch name to clipboard
Feb 10, 2026
Merged

fix: some bugs in DNS proxy#201
aarond10 merged 1 commit into
aarond10:masteraarond10/https_dns_proxy:masterfrom
toxeh:mastertoxeh/https_dns_proxy:masterCopy head branch name to clipboard

Conversation

@toxeh

@toxeh toxeh commented Feb 9, 2026

Copy link
Copy Markdown
Contributor
  • main.c: fix NULL pointer dereference in https_resp_cb() — req was used before NULL check, and FLOG also dereferenced NULL req->tx_id
  • main.c: fix potential crash in get_host_from_uri() — strlen(host) was called before checking curl_url_get() return code, host could be NULL
  • dns_server_tcp.c: fix data corruption on EAGAIN — sent += len was executed with len == -1 when EAGAIN/EWOULDBLOCK occurred, causing negative offset; added continue to skip the addition
  • dns_server_tcp.c: fix format string mismatches in FLOG() calls — format had 2 specifiers but 4 arguments were passed (ipstr, port, strerror, errno); changed format to include all 4 arguments
  • dns_server_tcp.c: fix typo 'listaning' -> 'listening'
  • https_client.c: remove deprecated CURLPIPE_HTTP1 (since libcurl 7.62.0), keep only CURLPIPE_MULTIPLEX

- main.c: fix NULL pointer dereference in https_resp_cb() — req was
  used before NULL check, and FLOG also dereferenced NULL req->tx_id
- main.c: fix potential crash in get_host_from_uri() — strlen(host)
  was called before checking curl_url_get() return code, host could
  be NULL
- dns_server_tcp.c: fix data corruption on EAGAIN — sent += len was
  executed with len == -1 when EAGAIN/EWOULDBLOCK occurred, causing
  negative offset; added continue to skip the addition
- dns_server_tcp.c: fix format string mismatches in FLOG() calls —
  format had 2 specifiers but 4 arguments were passed (ipstr, port,
  strerror, errno); changed format to include all 4 arguments
- dns_server_tcp.c: fix typo 'listaning' -> 'listening'
- https_client.c: remove deprecated CURLPIPE_HTTP1 (since libcurl
  7.62.0), keep only CURLPIPE_MULTIPLEX
@aarond10 aarond10 merged commit caad1eb into aarond10:master Feb 10, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.