~dlundquist/sniproxy/master : changes

~dlundquist/sniproxy/master » Changes from revision 413

From Revision 393 to 374

Rev	Summary	Authors	Tags	Date
393	Revert "Fix implicit signed conversion and sign comparison" Revert "Fix implicit signed conversion and sign comparison" This reverts commit b4e83367d2de00c1c74facd0bcc6592d0a26a6ba. This introduced regressions in the TLS module as reported in GitHub issue #271. Reverting until this can be examined in more detail.	Dustin Lundquist		6 years ago
392	Merge branches 'signed-conversion' and 'ipv6-v6only' Merge branches 'signed-conversion' and 'ipv6-v6only'	Dustin Lundquist		6 years ago
391	Fix implicit signed conversion and sign comparison Fix implicit signed conversion and sign comparison Compiled with clang's -Wpointer-sign -Wsign-conversion -Wsign-compare -Wsign-promo flags, and fixed all reported implicit signed conversions and comparison of signed and unsigned values. Most of these where resolved by adding explicit type casts, but manually reviewed context of this conversion to ensure overflow/underflow conditions where detected in error cases. In buffer module added BUFFER_MAX_SIZE constant set to 1GiB, this is to avoid ssize_t overflow on 32 bit platforms.	Dustin Lundquist		6 years ago
390	Fix several clang analyze warnings Fix several clang analyze warnings src/address.c:90:9: warning: Size argument is greater than the length of the destination buffer strncpy(((struct sockaddr_un )&sa)->sun_path, ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 1 warning generated. src/config.c:591:17: warning: Dereference of null pointer (loaded from variable 'vector_ptr') vector_ptr = vector; ~~~~~~~~~~ ^ 1 warning generated. src/connection.c:561:9: warning: Value stored to 'result' is never read result = 0; ^ ~ src/connection.c:752:5: warning: Value stored to 'message_pos' is never read message_pos += snprintf(message_pos, message_end - message_pos, ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 2 warnings generated. src/sniproxy.c:225:17: warning: Access to field 'pw_gid' results in a dereference of a null pointer (loaded from variable 'user') gid_t gid = user->pw_gid; ^~~~~~~~~~~~ src/sniproxy.c:235:13: warning: Access to field 'gr_gid' results in a dereference of a null pointer (loaded from variable 'group') gid = group->gr_gid; ^~~~~~~~~~~~~ 2 warnings generated.	Dustin Lundquist		6 years ago
389	reuseport_test reuseport_test	Dustin Lundquist		6 years ago
388	Disable reuseport in example config Disable reuseport in example config TravisCI builds where failing since SO_REUSEPORT isn't present in their environment and config_test fails to parse the example config. Cleanup error message so they are more consistent.	Dustin Lundquist		6 years ago
387	Cleanup SO_REUSEPORT Cleanup SO_REUSEPORT Remove unused include in config.c. Move boolean_true and boolean_false arrays into parse_boolean(). Rework accept_listener_reuseport() to parse argument when SO_REUSEPORT is absent, but raise error in reuseport is enabled and SO_REUSEPORT isn't available.	Dustin Lundquist		6 years ago
386	More fixes on SO_REUSEPORT More fixes on SO_REUSEPORT	Новгородов Игорь		7 years ago
385	Make ifdef more specific Make ifdef more specific	Новгородов Игорь		7 years ago
384	Remove redundant include Remove redundant include	Новгородов Игорь		7 years ago
383	Rework SO_REUSEPORT support Rework SO_REUSEPORT support	Новгородов Игорь		7 years ago
382	Add #ifder checks for SO_REUSEPORT to allow build on kernels... Add #ifder checks for SO_REUSEPORT to allow build on kernels <3.9 and other OSes	Новгородов Игорь		7 years ago
381	Update man page and example config with reuseport descriptio... Update man page and example config with reuseport description	Новгородов Игорь		7 years ago
380	Add SO_REUSEPORT support to listener Add SO_REUSEPORT support to listener	Новгородов Игорь		7 years ago
379	accept4 needs SOCK_NONBLOCK as flag accept4 needs SOCK_NONBLOCK as flag	Sebastian Wiedenroth		7 years ago
378	0.5.0 release 0.5.0 release	Dustin Lundquist	0.5.0	7 years ago
377	tests: fix tests for Perl 5.24.1 tests: fix tests for Perl 5.24.1 Apparently the Perl module include path no longer includes the current directory and keys no longer works on a hash ref.	Dustin Lundquist		7 years ago
376	config: treat duplicate config directives as error config: treat duplicate config directives as error Previously the last configuration directive for username, pidfile, groupname and listener table overrode any previous configuration directive but we leaked memory. Now detect duplicate directive and treat this is a configuration file error.	Dustin Lundquist		7 years ago
375	Explicitly check for NULL for groupname Explicitly check for NULL for groupname	Pieter Lexis		7 years ago
374	Implement dropping group privileges Implement dropping group privileges	Pieter Lexis		7 years ago