summaryrefslogtreecommitdiff
path: root/ninfod
Commit message (Collapse)AuthorAgeFilesLines
* ninfod: remove unused variablesJan Synacek2016-03-021-4/+1
|
* make `getaddrinfo()` and `getnameinfo()` usage consistentPavel Šimerda2015-06-181-3/+3
| | | | | | | | | | | | | | | | Use modern name resolution functions and use them in a consistent manner. * Avoid obsolete `gethostbyname()`, `gethostbyname2()` and `gethostbyaddr()` * Maintain consistency in variable names and error messages * Leave IDN handling up to name resolution functions Note: This patch provides a primitive translation of calls to obsolete functions to calls to modern functions but more changes are needed to process the whole result lists instead of using just the first result. Note: This patch skips functions that use jumps to avoid making more damage than benefits.
* ninfod: libgcrypt support.YOSHIFUJI Hideaki2014-10-255-33/+107
| | | | Signed-off-by: YOSHIFUJI Hideaki <hideaki.yoshifuji@miraclelinux.com>
* ninfod: Regenerate configure by autoconf-2.69.YOSHIFUJI Hideaki2014-10-251-101/+107
| | | | Signed-off-by: YOSHIFUJI Hideaki <hideaki.yoshifuji@miraclelinux.com>
* ninfod: Fix more unused variables.Jan Synacek2014-04-251-7/+1
| | | | | | | | | | ni_ifaddrs.c: In function ‘ni_ifaddrs’: | ni_ifaddrs.c:389:14: warning: variable ‘nlm_scope’ set but not used [-Wunused-but-set-variable] | ni_ifaddrs.c:353:13: warning: variable ‘ifflist’ set but not used [-Wunused-but-set-variable] | ni_ifaddrs.c:322:6: warning: variable ‘result’ set but not used [-Wunused-but-set-variable] Signed-off-by: Jan Synacek <jsynacek@redhat.com>
* ninfod: Open pidfile exclusively for write.YOSHIFUJI Hideaki2013-02-151-1/+18
| | | | Signed-off-by: YOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org>
* ninfod: Clean up signal/logfile handlingYOSHIFUJI Hideaki2013-02-151-34/+52
| | | | | | | | | - Do real work outside the signal handler. - Setup logfile early. - Do not try unlinking pid file if filename is NULL (this is not a bug, because filename is alway set non-NULL by default.) Signed-off-by: YOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org>
* ninfod: Allow printing usage without permission errors.YOSHIFUJI Hideaki2012-12-211-28/+29
| | | | Signed-off-by: YOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org>
* ninfod: Support GNU TLS.YOSHIFUJI Hideaki2012-12-214-8/+114
| | | | Signed-off-by: YOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org>
* ninfod: Do not expose freed buffer to caller.YOSHIFUJI Hideaki2012-12-201-0/+1
| | | | | | | | | | This is safe because we return an error to the caller, and the caller does not use the buffer. But it is better not to expose such a invalid pointer to caller. Bug noticed by Jan Synacek <jsynacek@redhat.com>. Signed-off-by: YOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org>
* ninfod: Fix off-by-one error to check possible programming error (again).YOSHIFUJI Hideaki2012-12-201-1/+1
| | | | | | | | | | | This bug was once fixed by commit 70bcae85 ("ninfod: Fix off-by-one error to check possible programming error."), but unfotunately, it was re-introduced by commit 9a2a2664 ("ninfod: Introduce ARRAY_SIZE macro for counting number of elements in an array."). Bug noticed by Jan Synacek <jsynacek@redhat.com>. Signed-off-by: YOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org>
* ninfod: Fix memory leakage in error path.YOSHIFUJI Hideaki2012-12-201-0/+1
| | | | | | Bug noticed by Jan Synacek <jsynacek@redhat.com>. Signed-off-by: YOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org>
* ninfod: Fix several warnings on ununsed variables.YOSHIFUJI Hideaki2012-12-072-6/+1
| | | | | | | | | | | | | | ninfod_core.c: In function 'pr_nodeinfo_noop': | ninfod_core.c:244:25: warning: unused variable 'replybuf' [-Wunused-variable] | ninfod_core.c: In function 'pr_nodeinfo_suptypes': | ninfod_core.c:273:9: warning: unused variable 'replylen' [-Wunused-variable] | ninfod.c: In function 'init_sock': | ninfod.c:255:6: warning: unused variable 'i' [-Wunused-variable] | ninfod.c: In function 'main': | ninfod.c:636:6: warning: unused variable 'c' [-Wunused-variable] Signed-off-by: YOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org>
* ninfod: Too many arguments for syslog(3)/fprintf(3) via DEBUG().YOSHIFUJI Hideaki2012-12-071-1/+1
| | | | | | | | | | ninfod.c: In function 'do_daemonize': | ninfod.c:419:6: warning: too many arguments for format [-Wformat-extra-args] Patch based on iputils-20121125-ninfod-minor.patch from Fedora. Signed-off-by: YOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org>
* ninfod: nanosleep(3) needs <time.h>.YOSHIFUJI Hideaki2012-12-071-0/+11
| | | | | | | | | | ninfod.c: In function 'ni_send': | ninfod.c:363:4: warning: implicit declaration of function 'nanosleep' [-Wimplicit-function-declaration] Patch based on iputils-20121125-ninfod-minor.patch from Fedora. Signed-off-by: YOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org>
* ninfod: Use unsigned int for digest.YOSHIFUJI Hideaki2012-12-071-1/+1
| | | | | | | | | Fixes following warning: | ninfod_name.c: In function 'nodeinfo_group': | ninfod_name.c:270:2: warning: pointer targets in passing argument 1 of 'MD5_Final' differ in signedness [-Wpointer-sign] | /usr/include/openssl/md5.h:113:5: note: expected 'unsigned char *' but argument is of type 'char *' Signed-off-by: YOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org>
* ninfod: Employ internal buffer in stderrlog() for common case.YOSHIFUJI Hideaki2012-11-251-5/+6
| | | | Signed-off-by: YOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org>
* ninfod: Do not mix output from multiple threads.YOSHIFUJI Hideaki2012-11-254-2/+54
| | | | | | | | - setbuf(stderr, NULL) - write a line at once (by vsnprintf + fputs); - No expcicit locks. Signed-off-by: YOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org>
* ninfod: Fix double-free without pthreads.YOSHIFUJI Hideaki2012-11-251-0/+1
| | | | Signed-off-by: YOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org>
* ninfod: Normalize timespec for delay.YOSHIFUJI Hideaki2012-11-251-2/+2
| | | | Signed-off-by: YOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org>
* ninfod: Apply default policy to refuse queries from global addresses.YOSHIFUJI Hideaki2012-11-251-1/+33
| | | | Signed-off-by: YOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org>
* ninfod: Discard multicat packet outside linklocal scope.YOSHIFUJI Hideaki2012-11-253-0/+32
| | | | Signed-off-by: YOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org>
* ninfod Makefile: More precise dependencies.YOSHIFUJI Hideaki2012-11-251-2/+10
| | | | Signed-off-by: YOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org>
* ninfod: Do noy try to memcpy to self.YOSHIFUJI Hideaki2012-11-251-2/+0
| | | | Signed-off-by: YOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org>
* ninfod: Fix usage message.YOSHIFUJI Hideaki2012-11-141-1/+1
| | | | | | -d means debug, not daemonize. Signed-off-by: YOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org>
* ninfod: Add run as user (-u user) option.YOSHIFUJI Hideaki2012-11-144-4/+145
| | | | | | | Note that we will not write pid file as root anymore, even if ninfod is suid-root'ed or if ninfod is run by root (with -u option). Signed-off-by: YOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org>
* ninfod: libcap support to drop capabilities.YOSHIFUJI Hideaki2012-11-144-2/+69
| | | | Signed-off-by: YOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org>
* ninfod: Add configure.YOSHIFUJI Hideaki2012-11-141-0/+5341
| | | | | | | 13e5b391.. ("ninfod: Attatch configure and renew config.h.in.") did not introduce configure in fact. Signed-off-by: YOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org>
* ninfod: Attatch configure and renew config.h.in.YOSHIFUJI Hideaki2012-10-121-4/+18
| | | | Signed-off-by: YOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org>
* ninfod: Delete ninfod.sh by make distclean, not by make clean.YOSHIFUJI Hideaki2012-10-111-1/+2
| | | | | | | ninfod.sh is generated by configure, so clean by "make distclean", not by "make clean". Signed-off-by: YOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org>
* ninfod: Introduce ARRAY_SIZE macro for counting number of elements in an array.YOSHIFUJI Hideaki2012-10-111-6/+8
| | | | Signed-off-by: YOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org>
* ninfod: Use __func__ instead of __FUNCTION__.YOSHIFUJI Hideaki2012-10-115-29/+29
| | | | Signed-off-by: YOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org>
* ninfod: Add datarootdir.YOSHIFUJI Hideaki2012-10-111-0/+1
| | | | | | Unused, but to suppress warning message from autoconf. Signed-off-by: YOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org>
* ninfod: Fix off-by-one error to check possible programming error.YOSHIFUJI Hideaki2012-10-111-1/+2
| | | | | | | This bug is not fatal at all because this check is only for possible programming error (which is not the case so far). Signed-off-by: YOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org>
* ninfod: Check return value of fscanf(3).YOSHIFUJI Hideaki2012-10-111-3/+7
| | | | Signed-off-by: YOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org>
* ninfod: Fix format string for string returned from strerror(3).YOSHIFUJI Hideaki2012-10-111-1/+1
| | | | Signed-off-by: YOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org>
* ninfod: Support newer environment supporting RFC3542.YOSHIFUJI Hideaki2012-10-111-9/+50
| | | | Signed-off-by: YOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org>
* ninfod: Add missing entry for ENABLE_SUPTYPES in config.h.in.YOSHIFUJI Hideaki2012-10-111-0/+3
| | | | Signed-off-by: YOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org>
* ninfod: Use %zu format string for size_t variable.YOSHIFUJI Hideaki2012-10-113-5/+5
| | | | Signed-off-by: YOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org>
* ninfod: struct in6_pktinfo requires -D_GNU_SOURCE.YOSHIFUJI Hideaki2012-10-111-1/+1
| | | | Signed-off-by: YOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org>
* ninfod: Node Information Query (RFC4620) daemon from USAGI Project.YOSHIFUJI Hideaki2012-10-1114-0/+3463
Signed-off-by: YOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org>