This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] Added missing ICMPv6 flags and option


On 02/12/2014 07:35 PM, Dan Luedtke wrote:

+	* inet/netinet/icmp6.h: Added missing Router Advertisement Flags
+	* inet/netinet/icmp6.h: Added ICMPv6 Recursive DNS Server Option

This entry does not follow the usual format. You should list the identifiers you changed.

+#define ND_RA_RTPREF_LOW         0x18
+#define ND_RA_RTPREF_MEDIUM      0x00
+#define ND_RA_RTPREF_HIGH        0x08

This should reference RFC 4191.  Is there are source for names?

+#define ND_RA_FLAG_PROXY         0x04

RFC 4389, it appears.  Where does the name come from?

+#define ND_OPT_RDNSS			25

  struct nd_opt_prefix_info     /* prefix information */
    {
@@ -231,6 +236,15 @@ struct nd_opt_mtu             /* MTU option */
      uint32_t  nd_opt_mtu_mtu;
    };

+struct nd_opt_rdnss           /* RDNSS option */
+  {
+    uint8_t   nd_opt_rdnss_type;
+    uint8_t   nd_opt_rdnss_len;
+    uint16_t  nd_opt_rdnss_reserved;
+    uint32_t  nd_opt_rdnss_lifetime;
+    /* followed by recursive DNS servers */
+  };

This seems to come from RFC 5006.  The names appear to be somewhat standard.

--
Florian Weimer / Red Hat Product Security Team


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]