Mercurial > hg > nginx
annotate src/core/ngx_sha1.h @ 9179:ea1f29c2010c
HTTP/2: fixed buffer management with HTTP/2 auto-detection.
As part of normal HTTP/2 processing, incomplete frames are saved in the
control state using a fixed size memcpy of NGX_HTTP_V2_STATE_BUFFER_SIZE.
For this matter, two state buffers are reserved in the HTTP/2 recv buffer.
As part of HTTP/2 auto-detection on plain TCP connections, initial data
is first read into a buffer specified by the client_header_buffer_size
directive that doesn't have state reservation. Previously, this made it
possible to over-read the buffer as part of saving the state.
The fix is to read the available buffer size rather than a fixed size.
Although memcpy of a fixed size can produce a better optimized code,
handling of incomplete frames isn't a common execution path, so it was
sacrificed for the sake of simplicity of the fix.
author | Sergey Kandaurov <pluknet@nginx.com> |
---|---|
date | Sat, 21 Oct 2023 18:48:24 +0400 |
parents | 9eefb38f0005 |
children |
rev | line source |
---|---|
1573 | 1 |
2 /* | |
3 * Copyright (C) Igor Sysoev | |
4412 | 4 * Copyright (C) Nginx, Inc. |
1573 | 5 */ |
6 | |
7 | |
8 #ifndef _NGX_SHA1_H_INCLUDED_ | |
9 #define _NGX_SHA1_H_INCLUDED_ | |
10 | |
11 | |
12 #include <ngx_config.h> | |
13 #include <ngx_core.h> | |
14 | |
15 | |
6586
1064ea81ed3a
An internal SHA1 implementation.
Maxim Dounin <mdounin@mdounin.ru>
parents:
4412
diff
changeset
|
16 typedef struct { |
1064ea81ed3a
An internal SHA1 implementation.
Maxim Dounin <mdounin@mdounin.ru>
parents:
4412
diff
changeset
|
17 uint64_t bytes; |
1064ea81ed3a
An internal SHA1 implementation.
Maxim Dounin <mdounin@mdounin.ru>
parents:
4412
diff
changeset
|
18 uint32_t a, b, c, d, e, f; |
1064ea81ed3a
An internal SHA1 implementation.
Maxim Dounin <mdounin@mdounin.ru>
parents:
4412
diff
changeset
|
19 u_char buffer[64]; |
1064ea81ed3a
An internal SHA1 implementation.
Maxim Dounin <mdounin@mdounin.ru>
parents:
4412
diff
changeset
|
20 } ngx_sha1_t; |
1064ea81ed3a
An internal SHA1 implementation.
Maxim Dounin <mdounin@mdounin.ru>
parents:
4412
diff
changeset
|
21 |
1064ea81ed3a
An internal SHA1 implementation.
Maxim Dounin <mdounin@mdounin.ru>
parents:
4412
diff
changeset
|
22 |
1064ea81ed3a
An internal SHA1 implementation.
Maxim Dounin <mdounin@mdounin.ru>
parents:
4412
diff
changeset
|
23 void ngx_sha1_init(ngx_sha1_t *ctx); |
1064ea81ed3a
An internal SHA1 implementation.
Maxim Dounin <mdounin@mdounin.ru>
parents:
4412
diff
changeset
|
24 void ngx_sha1_update(ngx_sha1_t *ctx, const void *data, size_t size); |
1064ea81ed3a
An internal SHA1 implementation.
Maxim Dounin <mdounin@mdounin.ru>
parents:
4412
diff
changeset
|
25 void ngx_sha1_final(u_char result[20], ngx_sha1_t *ctx); |
1064ea81ed3a
An internal SHA1 implementation.
Maxim Dounin <mdounin@mdounin.ru>
parents:
4412
diff
changeset
|
26 |
1064ea81ed3a
An internal SHA1 implementation.
Maxim Dounin <mdounin@mdounin.ru>
parents:
4412
diff
changeset
|
27 |
1573 | 28 #endif /* _NGX_SHA1_H_INCLUDED_ */ |