d74939d323
Add send_handshake_query helper function
...
Convert send_downenctest to use it.
Remove unused data/len args from send_downenctest.
2020-07-24 20:18:07 +02:00
a8a20f570e
Formatting: spaces around operators
2020-07-24 19:15:02 +02:00
aeacfbc2c0
Formatting: No spaces after '(' or before ')'
2020-07-24 19:02:41 +02:00
15f12d0693
Formatting: Space between 'for' and parenthesis
2020-07-24 18:57:57 +02:00
19d0d1be64
Formatting: Space between 'if' and parenthesis
2020-07-24 18:56:30 +02:00
310aedac8e
Formatting: no space before parenthesis in function call.
2020-07-24 18:51:05 +02:00
cde0b7632d
Set additional record count properly in NS reply
...
For when there is no IPv4 address to return.
2020-07-23 22:12:07 +02:00
ec6a1ac308
Fix IPv4 address in replies to A or NS queries (github issue #38 )
...
The destination field in struct query was changed from in_addr_t to
struct sockaddr_storage, wihtout updating the functions sending it
in src/dns.c.
Only add extra A answer for NS queries if destination refers to an
IPv4 address, and fail if trying to encode a reply to an A query
if destination is not IPv4.
This means NS requests received over IPv6 will not contain an address
and A requests will be ignored, unless the -n option is used, or the
www subdomain is requested which sets a fixed address (127.0.0.1).
2020-07-23 21:49:46 +02:00
d8f3ac8971
Add helper function tun_uses_header()
...
Remove code duplication and hide some ifdefs
2020-07-03 17:06:14 +02:00
07b2978326
Only check for utun if tun_device was given
...
Fixes github issue #37 hopefully.
2020-07-03 16:53:30 +02:00
2032b44949
Look up external IP via DNS instead of HTTP
...
Use myip.opendns.com via their resolver.
The code is now TCP free again :)
2020-05-24 16:06:41 +02:00
e8a4c66719
Clean up makefile
2020-05-24 14:12:40 +02:00
8379384d37
Improve usage/help text around finding external IP
2020-05-24 13:41:54 +02:00
d2f671bdc7
Define searchpath for route with macro
...
Allows it to be easily overridden using the compiler commmandline.
Just like IFCONFIGPATH.
2019-08-24 16:43:34 +02:00
37960ab9dd
client: warn when handshake fails due to BADIP
...
other parts of the code show the meaningful error message too, but
not at the spot where it happened for me.
2019-08-04 13:53:21 +02:00
d37ebdaca7
Merge pull request #28 from l29ah/strncat-warning
...
client.c: don't produce an useless strncat usage warning
2019-08-04 13:42:13 +02:00
024481c94b
Fix cross compilation
...
src/osflags hard codes the build architecture pkg-config. After making it
substitutable and substituting it from the build environment, iodine cross
builds successfully.
Original patch from Helmut Grohne <helmut@subdivi.de> in
https://bugs.debian.org/921822 , adjusted to Git HEAD.
2019-08-04 13:30:33 +02:00
b406009c6d
client.c: don't produce an useless strncat usage warning
2018-10-01 04:04:15 +03:00
99c0efc467
Try building working binaries for non-ancient Android
...
Based on help from admin@hypothermic.nl
2017-10-22 11:54:59 +02:00
72bdf7f20e
Merge branch 'protect-options' of https://github.com/Masaq-/iodine into protect_opts
2017-10-22 11:59:38 +02:00
122ac1a25d
check error returned by dns_decode
...
before this commit, sending "GET / HTTP/1.1" to server will cause uninitialized variable access.
2017-10-13 03:08:12 +08:00
79455c380d
consequently use tabs, and no spaces
...
and wrap lines at 80 characters
Signed-off-by: Ralf Ramsauer <ralf@ramses-pyramidenbau.de>
2017-04-10 21:26:13 +02:00
8c5127b375
don't zero-initialise variable
...
For global variables, the C standard ensures that this variable will be
zeroed on startup.
Signed-off-by: Ralf Ramsauer <ralf@ramses-pyramidenbau.de>
2017-03-11 21:30:50 -08:00
119d1b2da1
dns: improve code style
...
Signed-off-by: Ralf Ramsauer <ralf@ramses-pyramidenbau.de>
2017-03-11 21:30:50 -08:00
4591cafd27
encoding: simplify {places,eats}_dots
...
Why not using constant bools?
Much simpler than complex function calls, that eventually return
constant values.
Signed-off-by: Ralf Ramsauer <ralf@ramses-pyramidenbau.de>
2017-03-11 21:30:50 -08:00
0eb3b65158
encoding: use simple int's instead of accessor functions
...
Why are those values exposed to the outer world? They seem not be in use
anyway.
Signed-off-by: Ralf Ramsauer <ralf@ramses-pyramidenbau.de>
2017-03-11 21:30:50 -08:00
317511e3ca
nitpick: coding style
...
Signed-off-by: Ralf Ramsauer <ralf@ramses-pyramidenbau.de>
2017-03-11 21:30:50 -08:00
844a2798eb
encoder: use explicit variable names in operation structure
...
This makes it obvious, what each variable stands for.
Signed-off-by: Ralf Ramsauer <ralf@ramses-pyramidenbau.de>
2017-03-11 21:30:49 -08:00
6b438e7517
encoder: further simplifications
...
Get rid of unique header files for each encoder, consolidate them to
where they are actually needed: encoding.h.
This also simplifies the generation of the base64u decoder, as its
header file does not need to be generated any longer.
Signed-off-by: Ralf Ramsauer <ralf@ramses-pyramidenbau.de>
2017-03-11 21:30:49 -08:00
b517121f1c
base encoder: simplify structures
...
We don't need complex getters. Just expose basic operations as they are.
Signed-off-by: Ralf Ramsauer <ralf@ramses-pyramidenbau.de>
2017-03-11 21:30:49 -08:00
d05923d2b1
global: constify things
...
const everything, that should be const.
Signed-off-by: Ralf Ramsauer <ralf@ramses-pyramidenbau.de>
2017-03-11 21:30:49 -08:00
52c4940523
Warn, warn warn.
...
iodine does not seem to follow any styling guidelines (mixture of
different function prototypes, ...). So let's introduce some. This
will improve overall code quality and readability.
Additionally, warnings will improve code quality as well. Let's turn on
very pedantic warnings, and fix everything where the compiler barks
back.
Introduce the following function definition scheme:
type function_name(type name, type1 name1 ...)
{
}
This allows us to copy and paste the definition to the declaration by
selecting one single line.
Furthermore, limit line length to 80 characters.
Signed-off-by: Ralf Ramsauer <ralf@ramses-pyramidenbau.de>
2017-03-11 21:30:01 -08:00
ccc49f16f7
util: add missing #includes
...
Detected by activating stronger warnings. If include is missing,
prototypes might diverge.
Signed-off-by: Ralf Ramsauer <ralf@ramses-pyramidenbau.de>
2017-03-11 14:23:02 -08:00
ac6db12ddb
iodine, iodined: print intentional help to stdout
...
Like other unix tools: don't print application output, if the user asks
intentionally for help.
Signed-off-by: Ralf Ramsauer <ralf@ramses-pyramidenbau.de>
2017-03-11 10:39:38 -08:00
f8f87e3a54
iodine: remove spurious newline
...
Signed-off-by: Ralf Ramsauer <ralf@ramses-pyramidenbau.de>
2017-03-11 10:39:38 -08:00
84ded018cf
iodine: add usage for -4 and -6 command line arguments
...
Those arguments were introduced, but not documented.
Fixes: 619ede5d
2017-03-11 10:39:28 -08:00
5fd0874aba
iodine: improve help output format
...
Analogously to the patch for iodined before:
- remove redundant fprint calls
- maximum character width: 80 characters (improved readability)
- add additional newlines
Signed-off-by: Ralf Ramsauer <ralf@ramses-pyramidenbau.de>
2017-03-11 02:17:27 -08:00
43dad946c6
common: no need for \n in check_superuser
...
warnx adds the \n for us.
Signed-off-by: Ralf Ramsauer <ralf@ramses-pyramidenbau.de>
2017-03-11 02:17:27 -08:00
b884bfecfb
iodined: improve usage() readability
...
Best readability is at 80 characters maximum per line.
Signed-off-by: Ralf Ramsauer <ralf@ramses-pyramidenbau.de>
2017-03-11 02:17:27 -08:00
829a822de3
iodined: improve help output format
...
Standardise output format:
- remove redundant fprint calls
- maximum character width: 80 characters (improved readability)
- add additional newlines
Signed-off-by: Ralf Ramsauer <ralf@ramses-pyramidenbau.de>
2017-03-11 02:17:27 -08:00
fe0dbccbc5
iodine/iodined: get rid of redundant local __progname definition
...
Signed-off-by: Ralf Ramsauer <ralf@ramses-pyramidenbau.de>
2017-03-11 02:17:27 -08:00
2efa4dfb35
iodine: consolidate help() and usage()
...
This avoids redundancies.
Signed-off-by: Ralf Ramsauer <ralf@ramses-pyramidenbau.de>
2017-03-11 02:17:27 -08:00
1c86bf347f
iodine/iodined: do not print usage if no superuser
...
There is no value in printing the usage in this case, as the usage
doesn't give the user any hint on how to solve this issue.
Furthermore, replace the Windows implementation with an empty inline
function, which will result in no code.
Signed-off-by: Ralf Ramsauer <ralf@ramses-pyramidenbau.de>
2017-03-11 02:17:27 -08:00
c83d2ae03c
client: constify test patterns
...
There is no reason, why those patterns should not be const.
Signed-off-by: Ralf Ramsauer <ralf@ramses-pyramidenbau.de>
2017-03-11 02:17:27 -08:00
68443abd60
Fixed non-standard passing of va_list into functions declared with ...
2016-11-29 11:10:38 -08:00
1ad7c05b92
Do not validate the autodetected IPs of additional address families when using -4 (IPV4 only) and -6 (IPv6 only) flags
2016-11-23 01:00:11 +00:00
a96e2e7a69
Refuse attempts to set options after option negotiation has completed.
2016-10-22 04:46:48 +00:00
4987aa536d
Add support for socket activation by ipv6 socket
...
iodined may accept ipv4 and ipv6 sockets via systemd socket activation,
we need to figure out type of sockets.
2016-05-10 14:09:21 +03:00
7b1df75e3a
Fix compilation with systemd>=230 and for older systemd versions without compat-libs
2016-04-30 18:27:15 +02:00
a8a5fbbf0d
Second attempt at PIE binary for new android
...
Github PR #14
2015-08-05 20:04:03 +02:00
Erik Ekman
Daniel Schaefer
rofl0r
gregor herrmann
Sergey Alirzaev
chengzhicn
Ralf Ramsauer
Jes Bodi Klinke
Will Szumski
Masaq-
Aleksei Fedotov
Pascal Ernster