diff options
author | Christian Grothoff <christian@grothoff.org> | 2013-10-20 12:56:19 +0000 |
---|---|---|
committer | Christian Grothoff <christian@grothoff.org> | 2013-10-20 12:56:19 +0000 |
commit | 327ba8d91c668859873b586ffa999e79b02115f8 (patch) | |
tree | 7bda79fa8f8dc236d171ee03f94811b5b88b7cb2 /src/dns | |
parent | 2cbdb73ee49872f59151ab2ba6f1091a160f2ec7 (diff) | |
download | gnunet-327ba8d91c668859873b586ffa999e79b02115f8.tar.gz gnunet-327ba8d91c668859873b586ffa999e79b02115f8.zip |
-improve logging
Diffstat (limited to 'src/dns')
-rw-r--r-- | src/dns/dnsparser.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/src/dns/dnsparser.c b/src/dns/dnsparser.c index d10da1ca9..2f9fba0a6 100644 --- a/src/dns/dnsparser.c +++ b/src/dns/dnsparser.c | |||
@@ -294,7 +294,7 @@ parse_name (const char *udp_payload, | |||
294 | GNUNET_free (xstr); | 294 | GNUNET_free (xstr); |
295 | ret = tmp; | 295 | ret = tmp; |
296 | if (strlen (ret) > udp_payload_length) | 296 | if (strlen (ret) > udp_payload_length) |
297 | { | 297 | { |
298 | GNUNET_break_op (0); | 298 | GNUNET_break_op (0); |
299 | goto error; /* we are looping (building an infinite string) */ | 299 | goto error; /* we are looping (building an infinite string) */ |
300 | } | 300 | } |
@@ -796,7 +796,7 @@ GNUNET_DNSPARSER_builder_add_name (char *dst, | |||
796 | if (IDNA_SUCCESS != | 796 | if (IDNA_SUCCESS != |
797 | (rc = idna_to_ascii_8z (name, &idna_start, IDNA_ALLOW_UNASSIGNED))) | 797 | (rc = idna_to_ascii_8z (name, &idna_start, IDNA_ALLOW_UNASSIGNED))) |
798 | { | 798 | { |
799 | GNUNET_log (GNUNET_ERROR_TYPE_INFO, | 799 | GNUNET_log (GNUNET_ERROR_TYPE_WARNING, |
800 | _("Failed to convert UTF-8 name `%s' to DNS IDNA format: %s\n"), | 800 | _("Failed to convert UTF-8 name `%s' to DNS IDNA format: %s\n"), |
801 | name, | 801 | name, |
802 | idna_strerror (rc)); | 802 | idna_strerror (rc)); |
@@ -815,7 +815,10 @@ GNUNET_DNSPARSER_builder_add_name (char *dst, | |||
815 | else | 815 | else |
816 | len = dot - idna_name; | 816 | len = dot - idna_name; |
817 | if ( (len >= 64) || (0 == len) ) | 817 | if ( (len >= 64) || (0 == len) ) |
818 | { | ||
819 | GNUNET_break (0); | ||
818 | goto fail; /* segment too long or empty */ | 820 | goto fail; /* segment too long or empty */ |
821 | } | ||
819 | dst[pos++] = (char) (uint8_t) len; | 822 | dst[pos++] = (char) (uint8_t) len; |
820 | memcpy (&dst[pos], idna_name, len); | 823 | memcpy (&dst[pos], idna_name, len); |
821 | pos += len; | 824 | pos += len; |