- 20 Nov, 2020 3 commits
-
-
Reza Naghibi authored
-
Nils Goroll authored
closes #3379
-
Nils Goroll authored
avoiding local variables and VPI_BundleStrands() Fixes #3392
-
- 18 Nov, 2020 13 commits
-
-
Nils Goroll authored
-
Nils Goroll authored
I do not understand the global -esym does not work, but the warning is definitely bogus, the (unisgned)-1 argument is intentional and a documented feature.
-
Nils Goroll authored
we see this issue in two places (for libvcc and varnishd), hope this will silence both
-
Nils Goroll authored
-
Nils Goroll authored
it does not register use via tbl/sess_attr.h
-
Nils Goroll authored
leaving these in the "to be reviewed" section because I am not entirely sure. I think these cases are: - ses_set_attr / beresp_filter_fixed: flexelint bug? Looks like it does not register use via macros - ObjGetU32/Lck_DestroyClass API - HTTP_IterHdrPack / SYMTAB_NOERR cases we do not want to make static by design
-
Nils Goroll authored
-
Nils Goroll authored
Solve comparison between unsigned and -1 by moving a cast to where we need it. the existing code already implied that a VCL_INT be large enough to hold uid_t / gid_t.
-
Nils Goroll authored
Ref 31164908 Fixes #3243
-
Nils Goroll authored
-
Nils Goroll authored
we do want to use FLAGX | FLAGY even if zero for clarity error: Info 835: A zero has been given as right argument to operator '|'
-
Nils Goroll authored
Another case of struct instance names which we want to have, even though we do not currently use them.
-
Nils Goroll authored
Yes, we do not actually use the mode name, but we still want to have it for debugging and clarity. Info 754: local struct member 'vsc_sf_mode::name' (line 57, file ../../lib/libvarnishapi/vsc.c) not referenced
-
- 16 Nov, 2020 2 commits
-
-
Nils Goroll authored
Fixes #3243
-
Nils Goroll authored
-
- 13 Nov, 2020 5 commits
-
-
Dridi Boukelmoune authored
We want to avoid unsigned integers for reference counting since they would hide underflows.
-
Dridi Boukelmoune authored
Seeing a test failure on vtest I think there might be a race here that I hope to make more explicit next time it occurs.
-
Dridi Boukelmoune authored
-
Dridi Boukelmoune authored
A nice and tidy single token to integrate with the parameters table. Refs #3250
-
Dridi Boukelmoune authored
This reverts commit 5367be3f.
-
- 12 Nov, 2020 6 commits
-
-
Nils Goroll authored
polish differently and more sensibly
-
Nils Goroll authored
constify and avoid (char *) = strchr((const char *), x) error: vtc_varnish.c 559 Error 158: Assignment to variable 'la' (line 548) increases capability
-
Nils Goroll authored
as copies of the respective client side flags
-
Nils Goroll authored
-
Nils Goroll authored
-
Nils Goroll authored
When a parameter set is associated at the director level, it should be in effect with resolve=NOW as with resolve=LAZY.
-
- 10 Nov, 2020 1 commit
-
-
Poul-Henning Kamp authored
-
- 09 Nov, 2020 8 commits
-
-
Martin Blix Grydeland authored
This adds VTCP_Assert() on the result of read and write calls that deals with TCP sockets.
-
Martin Blix Grydeland authored
-
Martin Blix Grydeland authored
When used to check the result of read() and write() calls, it is useful that a positive return value is accepted in VTCP_Check().
-
Martin Blix Grydeland authored
Now that VTCP_Assert() accepts EAGAIN as a legal errno value for read() errors, uncomment this check.
-
Martin Blix Grydeland authored
Since the input value is sometimes the result of a read()/write() call, avoid truncating the ssize_t value on calling it.
-
Martin Blix Grydeland authored
When a socket timeout is set on a socket and the timeout expires, read() and write() calls on that socket will return (-1) with errno set to EAGAIN/EWOULDBLOCK.
-
Martin Blix Grydeland authored
Consistently use VTCP_Assert when asserting on the result of VTCP_Check().
-
Poul-Henning Kamp authored
-
- 06 Nov, 2020 1 commit
-
-
Poul-Henning Kamp authored
-
- 05 Nov, 2020 1 commit
-
-
Nils Goroll authored
for VCL_* functions from cache_varnishd.h
-