sssd.git/src/resolv, branch master

resolv: Use the same default timeout for SRV queries as previously

2015-03-03T15:15:28+00:00

When we changed the resolver code to use the TTL values from the DNS
queries instead of harcoded ones, we changed the default value by
accident.

Add a separate SRV TTL that is backwards-compatible with the old
harcoded value.

Reviewed-by: Pavel Březina

resolv: Fix a typo

2015-02-13T17:30:09+00:00

Reviewed-by: Pavel Reichl

RESOLV: Remove obsolete in-tree implementation of SRV and TXT parsing

2015-02-11T10:12:04+00:00

SSSD contained several backwards-compatible definitions of SRV and TXT
APIs as well as structures that carry TTL data. These were intended for
RHEL-5 and older releases. Since we don't support those upstream, it's
better to remove the code -- it has drifted apart from upstream anyway.

Reviewed-by: Pavel Březina

RESOLV: Add an internal function to read TTL from a DNS packet

2015-02-11T10:03:34+00:00

Related:
https://fedorahosted.org/sssd/ticket/1884

Adds an internal resolver function that reads the TTL for SRV records as
specified by RFC-2181. Several internal c-ares definitions are used
until c-ares contains a function that exposes all this information via a
parsing function.

Reviewed-by: Pavel Březina

resolv: fix server sort by weight

2014-07-14T13:05:04+00:00

When the server list consist only from servers with zero weight the
output list contained only one server.

Resolves:
https://fedorahosted.org/sssd/ticket/2357

Reviewed-by: Pavel Reichl

Unify usage of function gethostname

2014-06-03T12:36:42+00:00

man gethostanme says:
NOTES
   SUSv2  guarantees  that  "Host  names  are  limited  to   255   bytes".
   POSIX.1-2001 guarantees that "Host names (not including the terminating
   null  byte)  are  limited   to   HOST_NAME_MAX   bytes".    On   Linux,
   HOST_NAME_MAX  is  defined  with the value 64, which has been the limit
   since Linux 1.0 (earlier kernels imposed a limit of 8 bytes).

Reviewed-by: Pavel Březina

resolv: Do not try to free addrinfo in case of error

2014-05-20T19:07:35+00:00

We should call freeaddrinfo only in situation if getaddrinfo succeeds.
It is not specified in RFC2553 the behaviour of calling freeaddrinfo
with the NULL argument. It is more portable to do not rely on undocumented
behaviour of glibc.

http://www.ietf.org/rfc/rfc2553.txt [Page 28]

Reviewed-by: Jakub Hrozek

Remove unused argument from resolv_gethostbyname_dns_parse

2014-04-07T15:46:15+00:00

Reviewed-by: Pavel Reichl

resolv_gethostbyname_dns_parse(): remove tmp_ctx

2014-02-17T16:39:48+00:00

Resolves:
https://fedorahosted.org/sssd/ticket/2198

Reviewed-by: Lukáš Slebodník

Update DEBUG* invocations to use new levels

2014-02-12T21:31:02+00:00

Use a script to update DEBUG* macro invocations, which use literal
numbers for levels, to use bitmask macros instead:

grep -rl --include '*.[hc]' DEBUG . |
    while read f; do
        mv "$f"{,.orig}
        perl -e 'use strict;
                 use File::Slurp;
                 my @map=qw"
                    SSSDBG_FATAL_FAILURE
                    SSSDBG_CRIT_FAILURE
                    SSSDBG_OP_FAILURE
                    SSSDBG_MINOR_FAILURE
                    SSSDBG_CONF_SETTINGS
                    SSSDBG_FUNC_DATA
                    SSSDBG_TRACE_FUNC
                    SSSDBG_TRACE_LIBS
                    SSSDBG_TRACE_INTERNAL
                    SSSDBG_TRACE_ALL
                 ";
                 my $text=read_file(\*STDIN);
                 my $repl;
                 $text=~s/
                            ^
                            (
                                .*
                                \b
                                (DEBUG|DEBUG_PAM_DATA|DEBUG_GR_MEM)
                                \s*
                                \(\s*
                            )(
                                [0-9]
                            )(
                                \s*,
                            )
                            (
                                \s*
                            )
                            (
                                .*
                            )
                            $
                         /
                            $repl = $1.$map[$3].$4.$5.$6,
                            length($repl) <= 80
                                ? $repl
                                : $1.$map[$3].$4."\n".(" " x length($1)).$6
                         /xmge;
                 print $text;
        ' < "$f.orig" > "$f"
        rm "$f.orig"
    done

Reviewed-by: Jakub Hrozek 
Reviewed-by: Stephen Gallagher 
Reviewed-by: Simo Sorce