/* @todo: this is a hack */
# define DEBUG if (fr_debug_flag && fr_log_fp) fr_printf_log
-void fr_strerror_printf(const char *fmt, ...);
# define debug_pair(vp) do { if (fr_debug_flag && fr_log_fp) { \
vp_print(fr_log_fp, vp); \
} \
* INADDR_ANY : 68 -> INADDR_BROADCAST : 67 REQUEST
* INADDR_BROADCAST : 68 <- SERVER_IP : 67 ACK
*/
-static const char *dhcp_header_names[] = {
+static char const *dhcp_header_names[] = {
"DHCP-Opcode",
"DHCP-Hardware-Type",
"DHCP-Hardware-Address-Length",
NULL
};
-static const char *dhcp_message_types[] = {
+static char const *dhcp_message_types[] = {
"invalid",
"DHCP-Discover",
"DHCP-Offer",
int overload = 0;
int field = DHCP_OPTION_FIELD;
size_t where, size;
- uint8_t *data = packet->options;
+ uint8_t *data;
where = 0;
size = packet_size - offsetof(dhcp_packet_t, options);
socklen_t sizeof_src;
socklen_t sizeof_dst;
RADIUS_PACKET *packet;
- int port;
+ uint16_t port;
uint8_t *code;
+ ssize_t data_len;
- packet = rad_alloc(NULL, 0);
+ packet = rad_alloc(NULL, false);
if (!packet) {
fr_strerror_printf("Failed allocating packet");
return NULL;
sizeof_src = sizeof(src);
#ifdef WITH_UDPFROMTO
sizeof_dst = sizeof(dst);
- packet->data_len = recvfromto(sockfd, packet->data, MAX_PACKET_SIZE, 0,
- (struct sockaddr *)&src, &sizeof_src,
- (struct sockaddr *)&dst, &sizeof_dst);
+ data_len = recvfromto(sockfd, packet->data, MAX_PACKET_SIZE, 0,
+ (struct sockaddr *)&src, &sizeof_src,
+ (struct sockaddr *)&dst, &sizeof_dst);
#else
- packet->data_len = recvfrom(sockfd, packet->data, MAX_PACKET_SIZE, 0,
- (struct sockaddr *)&src, &sizeof_src);
+ data_len = recvfrom(sockfd, packet->data, MAX_PACKET_SIZE, 0,
+ (struct sockaddr *)&src, &sizeof_src);
#endif
- if (packet->data_len <= 0) {
- fr_strerror_printf("Failed reading DHCP socket: %s", strerror(errno));
+ if (data_len <= 0) {
+ fr_strerror_printf("Failed reading DHCP socket: %s", fr_syserror(errno));
rad_free(&packet);
return NULL;
}
+ packet->data_len = data_len;
if (packet->data_len < MIN_PACKET_SIZE) {
fr_strerror_printf("DHCP packet is too small (%zu < %d)",
- packet->data_len, MIN_PACKET_SIZE);
+ packet->data_len, MIN_PACKET_SIZE);
+ rad_free(&packet);
+ return NULL;
+ }
+
+ if (packet->data_len > MAX_PACKET_SIZE) {
+ fr_strerror_printf("DHCP packet is too large (%zx > %d)",
+ packet->data_len, MAX_PACKET_SIZE);
rad_free(&packet);
return NULL;
}
* This should never fail...
*/
if (getsockname(sockfd, (struct sockaddr *) &dst, &sizeof_dst) < 0) {
- fr_strerror_printf("getsockname failed: %s", strerror(errno));
+ fr_strerror_printf("getsockname failed: %s", fr_syserror(errno));
rad_free(&packet);
- return NULL;
+ return NULL;
}
#endif
if (fr_debug_flag > 1) {
char type_buf[64];
- const char *name = type_buf;
+ char const *name = type_buf;
char src_ip_buf[256], dst_ip_buf[256];
if ((packet->code >= PW_DHCP_DISCOVER) &&
packet->code - PW_DHCP_OFFSET);
}
- DEBUG("Received %s of id %08x from %s:%d to %s:%d\n",
+ DEBUG("Received %s of Id %08x from %s:%d to %s:%d\n",
name, (unsigned int) packet->id,
inet_ntop(packet->src_ipaddr.af,
&packet->src_ipaddr.ipaddr,
fr_ipaddr2sockaddr(&packet->dst_ipaddr, packet->dst_port,
&dst, &sizeof_dst);
+ if (packet->data_len == 0) {
+ fr_strerror_printf("No data to send");
+ return -1;
+ }
+
if (fr_debug_flag > 1) {
char type_buf[64];
- const char *name = type_buf;
+ char const *name = type_buf;
#ifdef WITH_UDPFROMTO
- char src_ip_buf[256];
+ char src_ip_buf[INET6_ADDRSTRLEN];
#endif
- char dst_ip_buf[256];
+ char dst_ip_buf[INET6_ADDRSTRLEN];
if ((packet->code >= PW_DHCP_DISCOVER) &&
(packet->code <= PW_DHCP_INFORM)) {
DEBUG(
#ifdef WITH_UDPFROMTO
- "Sending %s of id %08x from %s:%d to %s:%d\n",
+ "Sending %s Id %08x from %s:%d to %s:%d\n",
#else
- "Sending %s of id %08x to %s:%d\n",
+ "Sending %s Id %08x to %s:%d\n",
#endif
name, (unsigned int) packet->id,
#ifdef WITH_UDPFROMTO
- inet_ntop(packet->src_ipaddr.af,
- &packet->src_ipaddr.ipaddr,
- src_ip_buf, sizeof(src_ip_buf)),
+ inet_ntop(packet->src_ipaddr.af, &packet->src_ipaddr.ipaddr, src_ip_buf, sizeof(src_ip_buf)),
packet->src_port,
#endif
- inet_ntop(packet->dst_ipaddr.af,
- &packet->dst_ipaddr.ipaddr,
- dst_ip_buf, sizeof(dst_ip_buf)),
+ inet_ntop(packet->dst_ipaddr.af, &packet->dst_ipaddr.ipaddr, dst_ip_buf, sizeof(dst_ip_buf)),
packet->dst_port);
}
#endif
}
-static int fr_dhcp_attr2vp(RADIUS_PACKET *packet, VALUE_PAIR *vp,
- const uint8_t *p, size_t alen);
+static int fr_dhcp_attr2vp(RADIUS_PACKET *packet, VALUE_PAIR *vp, uint8_t const *p, size_t alen);
-static int decode_tlv(RADIUS_PACKET *packet, VALUE_PAIR *tlv,
- const uint8_t *data, size_t data_len)
+static int fr_dhcp_decode_suboption(RADIUS_PACKET *packet, VALUE_PAIR *tlv, uint8_t const *data, size_t data_len)
{
- const uint8_t *p;
- VALUE_PAIR *head, **tail, *vp;
+ uint8_t const *p, *q;
+ VALUE_PAIR *head, *vp;
+ vp_cursor_t cursor;
/*
* Take a pass at parsing it.
*/
p = data;
- while (p < (data + data_len)) {
- if ((p + 2) > (data + data_len)) goto make_tlv;
+ q = data + data_len;
+ while (p < q) {
+ /*
+ * The RFC 3046 is very specific about not allowing termination
+ * with a 255 sub-option. But vendors are stupid, so allow it
+ * and the 0 padding sub-option.
+ * This requirement really should be a SHOULD anyway...
+ */
+ if (*p == 0) {
+ p++;
+ continue;
+ }
+ if (*p == 255) {
+ q--;
+ break;
+ }
- if ((p + p[1] + 2) > (data + data_len)) goto make_tlv;
- p += 2 + p[1];
+ /*
+ * Check if reading length would take us past the end of the buffer
+ */
+ if (++p >= q) goto malformed;
+ p += p[0];
+
+ /*
+ * Check if length > the length of the buffer we have left
+ */
+ if (p >= q) goto malformed;
+ p++;
}
/*
* Got here... must be well formed.
*/
head = NULL;
- tail = &head;
+ fr_cursor_init(&cursor, &head);
p = data;
- while (p < (data + data_len)) {
+ while (p < q) {
vp = paircreate(packet, tlv->da->attr | (p[0] << 8), DHCP_MAGIC_VENDOR);
if (!vp) {
pairfree(&head);
- goto make_tlv;
+ goto malformed;
}
if (fr_dhcp_attr2vp(packet, vp, p + 2, p[1]) < 0) {
pairfree(&head);
- goto make_tlv;
+ goto malformed;
}
- *tail = vp;
- tail = &(vp->next);
+ fr_cursor_insert(&cursor, vp);
p += 2 + p[1];
}
/*
* The caller allocated TLV, so we need to copy the FIRST
* attribute over top of that.
+ *
+ * This is a pretty awful hack, but we should be able to
+ * clean it up when we get nested VPs so lets leave it for
+ * now.
*/
if (head) {
+ /* Cleanup any old TLV data */
+ talloc_free(tlv->vp_tlv);
+
+ /* @fixme fragile */
memcpy(tlv, head, sizeof(*tlv));
- head->next = NULL;
- pairfree(&head);
+
+ /* If the VP has a talloced value we need to reparent it to the original TLV attribute */
+ switch (head->da->type) {
+ case PW_TYPE_STRING:
+ case PW_TYPE_OCTETS:
+ case PW_TYPE_TLV:
+ (void) talloc_steal(tlv, head->data.ptr);
+
+ default:
+ break;
+ }
+ tlv->next = head->next;
+ talloc_free(head);
}
return 0;
-make_tlv:
+malformed:
tlv->vp_tlv = talloc_array(tlv, uint8_t, data_len);
if (!tlv->vp_tlv) {
fr_strerror_printf("No memory");
/*
* Decode ONE value into a VP
*/
-static int fr_dhcp_attr2vp(RADIUS_PACKET *packet, VALUE_PAIR *vp,
- const uint8_t *p, size_t alen)
+static int fr_dhcp_attr2vp(RADIUS_PACKET *packet, VALUE_PAIR *vp, uint8_t const *p, size_t alen)
{
+ char *q;
+
switch (vp->da->type) {
case PW_TYPE_BYTE:
if (alen != 1) goto raw;
- vp->vp_integer = p[0];
+ vp->vp_byte = p[0];
break;
case PW_TYPE_SHORT:
if (alen != 2) goto raw;
- memcpy(&vp->vp_integer, p, 2);
- vp->vp_integer = ntohs(vp->vp_integer);
+ memcpy(&vp->vp_short, p, 2);
+ vp->vp_short = ntohs(vp->vp_short);
break;
case PW_TYPE_INTEGER:
vp->vp_integer = ntohl(vp->vp_integer);
break;
- case PW_TYPE_IPADDR:
+ case PW_TYPE_IPV4_ADDR:
if (alen != 4) goto raw;
/*
* Keep value in Network Order!
break;
case PW_TYPE_STRING:
- if (alen > 253) return -1;
- memcpy(vp->vp_strvalue, p , alen);
- vp->vp_strvalue[alen] = '\0';
+ vp->vp_strvalue = q = talloc_array(vp, char, alen + 1);
+ vp->type = VT_DATA;
+ memcpy(q, p , alen);
+ q[alen] = '\0';
+ break;
+
+ case PW_TYPE_ETHERNET:
+ memcpy(vp->vp_ether, p, sizeof(vp->vp_ether));
+ vp->length = sizeof(vp->vp_ether);
break;
-
+
/*
* Value doesn't match up with attribute type, overwrite the
* vp's original DICT_ATTR with an unknown one.
*/
raw:
if (pair2unknown(vp) < 0) return -1;
-
+
case PW_TYPE_OCTETS:
- if (alen > 253) return -1;
- memcpy(vp->vp_octets, p, alen);
+ if (alen > 255) return -1;
+ pairmemcpy(vp, p, alen);
break;
+ /*
+ * For option 82 et al...
+ */
case PW_TYPE_TLV:
- return decode_tlv(packet, vp, p, alen);
+ return fr_dhcp_decode_suboption(packet, vp, p, alen);
default:
fr_strerror_printf("Internal sanity check %d %d", vp->da->type, __LINE__);
}
ssize_t fr_dhcp_decode_options(RADIUS_PACKET *packet,
- uint8_t *data, size_t len, VALUE_PAIR **head)
+ uint8_t const *data, size_t len, VALUE_PAIR **head)
{
int i;
- VALUE_PAIR *vp, **tail;
- uint8_t *p, *next;
+ VALUE_PAIR *vp;
+ vp_cursor_t cursor;
+ uint8_t const *p, *next;
next = data;
*head = NULL;
- tail = head;
+ fr_cursor_init(&cursor, head);
+
/*
* FIXME: This should also check sname && file fields.
* See the dhcp_get_option() function above.
*/
while (next < (data + len)) {
int num_entries, alen;
- const DICT_ATTR *da;
-
+ DICT_ATTR const *da;
+
p = next;
- if (*p == 0) break;
- if (*p == 255) break; /* end of options signifier */
+ if (*p == 0) { /* 0x00 - Padding option */
+ next++;
+ continue;
+ }
+ if (*p == 255) break; /* 0xff - End of options signifier */
+
if ((p + 2) > (data + len)) break;
next = p + 2 + p[1];
- if (p[1] >= 253) {
- fr_strerror_printf("Attribute too long %u %u",
- p[0], p[1]);
- continue;
- }
-
da = dict_attrbyvalue(p[0], DHCP_MAGIC_VENDOR);
if (!da) {
- fr_strerror_printf("Attribute not in our dictionary: %u",
- p[0]);
+ fr_strerror_printf("Attribute not in our dictionary: %u", p[0]);
continue;
}
alen = 2;
break;
- case PW_TYPE_IPADDR:
+ case PW_TYPE_IPV4_ADDR:
case PW_TYPE_INTEGER:
case PW_TYPE_DATE: /* ignore any trailing data */
num_entries = alen >> 2;
for (i = 0; i < num_entries; i++) {
vp = pairmake(packet, NULL, da->name, NULL, T_OP_ADD);
if (!vp) {
- fr_strerror_printf("Cannot build attribute %s",
- fr_strerror());
+ fr_strerror_printf("Cannot build attribute %s", fr_strerror());
pairfree(head);
return -1;
}
-
- /*
- * Hack for ease of use.
- */
- if ((da->vendor == DHCP_MAGIC_VENDOR) &&
- (da->attr == 61) && !da->flags.array &&
- (alen == 7) && (*p == 1) && (num_entries == 1)) {
- memcpy(vp->vp_octets, p + 1, 6);
- vp->length = alen;
- } else if (fr_dhcp_attr2vp(packet, vp, p, alen) < 0) {
+ if (fr_dhcp_attr2vp(packet, vp, p, alen) < 0) {
pairfree(&vp);
pairfree(head);
return -1;
}
- *tail = vp;
- while (*tail) {
- debug_pair(*tail);
- tail = &(*tail)->next;
+ fr_cursor_insert(&cursor, vp);
+
+ for (vp = fr_cursor_current(&cursor);
+ vp;
+ vp = fr_cursor_next(&cursor)) {
+ debug_pair(vp);
}
p += alen;
} /* loop over array entries */
} /* loop over the entire packet */
-
+
return next - data;
}
size_t i;
uint8_t *p;
uint32_t giaddr;
- VALUE_PAIR *head, *vp, **tail;
+ vp_cursor_t cursor;
+ VALUE_PAIR *head = NULL, *vp;
VALUE_PAIR *maxms, *mtu;
- head = NULL;
- tail = &head;
+ fr_cursor_init(&cursor, &head);
p = packet->data;
if ((fr_debug_flag > 2) && fr_log_fp) {
* Decode the header.
*/
for (i = 0; i < 14; i++) {
+ char *q;
+
vp = pairmake(packet, NULL, dhcp_header_names[i], NULL, T_OP_EQ);
if (!vp) {
char buffer[256];
return -1;
}
- if ((i == 11) &&
- (packet->data[1] == 1) &&
- (packet->data[2] != 6)) {
- fr_strerror_printf("chaddr of incorrect length for ethernet");
+ /*
+ * If chaddr does != 6 bytes it's probably not ethernet, and we should store
+ * it as an opaque type (octets).
+ */
+ if ((i == 11) && (packet->data[1] == 1) && (packet->data[2] != sizeof(vp->vp_ether))) {
+ DICT_ATTR const *da = dict_attrunknown(vp->da->attr, vp->da->vendor, true);
+ if (!da) {
+ return -1;
+ }
+ vp->da = da;
}
switch (vp->da->type) {
case PW_TYPE_BYTE:
- vp->vp_integer = p[0];
+ vp->vp_byte = p[0];
vp->length = 1;
break;
case PW_TYPE_SHORT:
- vp->vp_integer = (p[0] << 8) | p[1];
+ vp->vp_short = (p[0] << 8) | p[1];
vp->length = 2;
break;
vp->length = 4;
break;
- case PW_TYPE_IPADDR:
+ case PW_TYPE_IPV4_ADDR:
memcpy(&vp->vp_ipaddr, p, 4);
vp->length = 4;
break;
case PW_TYPE_STRING:
- memcpy(vp->vp_strvalue, p, dhcp_header_sizes[i]);
- vp->vp_strvalue[dhcp_header_sizes[i]] = '\0';
+ vp->vp_strvalue = q = talloc_array(vp, char, dhcp_header_sizes[i] + 1);
+ vp->type = VT_DATA;
+ memcpy(q, p, dhcp_header_sizes[i]);
+ q[dhcp_header_sizes[i]] = '\0';
vp->length = strlen(vp->vp_strvalue);
if (vp->length == 0) {
pairfree(&vp);
break;
case PW_TYPE_OCTETS:
- memcpy(vp->vp_octets, p, packet->data[2]);
- vp->length = packet->data[2];
+ pairmemcpy(vp, p, packet->data[2]);
break;
case PW_TYPE_ETHERNET:
if (!vp) continue;
debug_pair(vp);
- *tail = vp;
- tail = &vp->next;
+ fr_cursor_insert(&cursor, vp);
}
/*
* Loop over the options.
*/
-
+
/*
* Nothing uses tail after this call, if it does in the future
* it'll need to find the new tail...
*/
- if (fr_dhcp_decode_options(packet,
- packet->data + 240, packet->data_len - 240,
- tail) < 0) {
- return -1;
+ {
+ VALUE_PAIR *options = NULL;
+
+ if (fr_dhcp_decode_options(packet,
+ packet->data + 240, packet->data_len - 240,
+ &options) < 0) {
+ return -1;
+ }
+
+ if (options) {
+ fr_cursor_insert(&cursor, options);
+ }
}
/*
mtu = pairfind(packet->vps, 26, DHCP_MAGIC_VENDOR, TAG_ANY);
if (mtu && (mtu->vp_integer < DEFAULT_PACKET_SIZE)) {
- fr_strerror_printf("DHCP Fatal: Client says MTU is smaller than minimum permitted by the specification.");
+ fr_strerror_printf("DHCP Fatal: Client says MTU is smaller than minimum permitted by the specification");
return -1;
}
maxms->vp_integer = mtu->vp_integer;
}
- if (fr_debug_flag > 0) {
- for (vp = packet->vps; vp != NULL; vp = vp->next) {
- /* Print debug output here? */
- }
- }
-
if (fr_debug_flag) fflush(stdout);
return 0;
}
-static int attr_cmp(const void *one, const void *two)
+int8_t fr_dhcp_attr_cmp(void const *a, void const *b)
{
- const VALUE_PAIR * const *a = one;
- const VALUE_PAIR * const *b = two;
+ VALUE_PAIR const *my_a = a;
+ VALUE_PAIR const *my_b = b;
+
+ VERIFY_VP(my_a);
+ VERIFY_VP(my_b);
/*
* DHCP-Message-Type is first, for simplicity.
*/
- if (((*a)->da->attr == 53) &&
- (*b)->da->attr != 53) return -1;
+ if ((my_a->da->attr == 53) && (my_b->da->attr != 53)) return -1;
/*
* Relay-Agent is last
*/
- if (((*a)->da->attr == 82) &&
- (*b)->da->attr != 82) return 1;
+ if ((my_a->da->attr == 82) && (my_b->da->attr != 82)) return 1;
- return ((*a)->da->attr - (*b)->da->attr);
+ if (my_a->da->attr < my_b->da->attr) return -1;
+ if (my_a->da->attr > my_b->da->attr) return 1;
+
+ return 0;
}
-/*
- * @fixme Check room!
+/** Write DHCP option value into buffer
+ *
+ * Does not include DHCP option length or number.
+ *
+ * @param out where to write the DHCP option.
+ * @param outlen length of output buffer.
+ * @param vp option to encode.
+ * @return the length of data writen, -1 if out of buffer, -2 if unsupported type.
*/
-static size_t fr_dhcp_vp2attr(VALUE_PAIR *vp, uint8_t *p, UNUSED size_t room)
+static ssize_t fr_dhcp_vp2attr(uint8_t *out, size_t outlen, VALUE_PAIR *vp)
{
- size_t length;
uint32_t lvalue;
+ uint8_t *p = out;
+
+ if (outlen < vp->length) {
+ return -1;
+ }
- /*
- * Search for all attributes of the same
- * type, and pack them into the same
- * attribute.
- */
switch (vp->da->type) {
case PW_TYPE_BYTE:
- length = 1;
- *p = vp->vp_integer & 0xff;
+ *p = vp->vp_byte;
break;
case PW_TYPE_SHORT:
- length = 2;
- p[0] = (vp->vp_integer >> 8) & 0xff;
- p[1] = vp->vp_integer & 0xff;
+ p[0] = (vp->vp_short >> 8) & 0xff;
+ p[1] = vp->vp_short & 0xff;
break;
case PW_TYPE_INTEGER:
- length = 4;
lvalue = htonl(vp->vp_integer);
memcpy(p, &lvalue, 4);
break;
- case PW_TYPE_IPADDR:
- length = 4;
+ case PW_TYPE_IPV4_ADDR:
memcpy(p, &vp->vp_ipaddr, 4);
break;
case PW_TYPE_ETHERNET:
- length = 6;
memcpy(p, &vp->vp_ether, 6);
break;
case PW_TYPE_STRING:
memcpy(p, vp->vp_strvalue, vp->length);
- length = vp->length;
break;
case PW_TYPE_TLV: /* FIXME: split it on 255? */
memcpy(p, vp->vp_tlv, vp->length);
- length = vp->length;
break;
case PW_TYPE_OCTETS:
memcpy(p, vp->vp_octets, vp->length);
- length = vp->length;
break;
default:
- fr_strerror_printf("BAD TYPE2 %d", vp->da->type);
- length = 0;
- break;
+ fr_strerror_printf("Unsupported option type %d", vp->da->type);
+ return -2;
}
- return length;
+ return vp->length;
}
-static VALUE_PAIR *fr_dhcp_vp2suboption(RADIUS_PACKET *packet, VALUE_PAIR *vps)
+/** Create a new TLV attribute from multiple sub options
+ *
+ * @param[in,out] ctx to allocate new attribute in.
+ * @param[in,out] cursor should be set to the start of the list of TLV attributes.
+ * Will be advanced to the first non-TLV attribute.
+ * @return attribute holding the concatenation of the values of the sub options.
+ */
+static VALUE_PAIR *fr_dhcp_vp2suboption(TALLOC_CTX *ctx, vp_cursor_t *cursor)
{
- int length;
- unsigned int attribute;
- uint8_t *ptr;
+ ssize_t length;
+ unsigned int parent; /* Parent attribute of suboption */
+ uint8_t attr = 0;
+ uint8_t *p, *opt_len = NULL;
+ vp_cursor_t to_pack;
VALUE_PAIR *vp, *tlv;
- attribute = vps->da->attr & 0xffff00ff;
+#define SUBOPTION_PARENT(_x) (_x & 0xffff00ff)
+#define SUBOPTION_ATTR(_x) ((_x & 0xff00) >> 8)
- tlv = paircreate(packet, attribute, DHCP_MAGIC_VENDOR);
+ vp = fr_cursor_current(cursor);
+ if (!vp) return NULL;
+
+ parent = SUBOPTION_PARENT(vp->da->attr);
+ tlv = paircreate(ctx, parent, DHCP_MAGIC_VENDOR);
if (!tlv) return NULL;
- tlv->length = 0;
- for (vp = vps; vp != NULL; vp = vp->next) {
+ fr_cursor_copy(&to_pack, cursor);
+
+ /*
+ * Loop over TLVs to determine how much memory we need to allocate
+ *
+ * We advanced the cursor we were passed, so if we fail encoding,
+ * the cursor is at the right position for the next potentially
+ * encodable attr.
+ */
+ for (vp = fr_cursor_current(cursor);
+ vp && vp->da->flags.is_tlv && !vp->da->flags.extended && (SUBOPTION_PARENT(vp->da->attr) == parent);
+ vp = fr_cursor_next(cursor)) {
/*
- * Group the attributes ONLY until we see a
- * non-TLV attribute.
+ * If it's not an array type or is an array type, but is not the same
+ * as the previous attribute, we add 2 for the additional sub-option
+ * header bytes.
*/
- if (!vp->da->flags.is_tlv ||
- vp->da->flags.extended ||
- ((vp->da->attr & 0xffff00ff) != attribute)) {
- break;
+ if (!vp->da->flags.array || (SUBOPTION_ATTR(vp->da->attr) != attr)) {
+ attr = SUBOPTION_ATTR(vp->da->attr);
+ tlv->length += 2;
}
-
- tlv->length += vp->length + 2;
- }
-
- if (!tlv->length) {
- pairfree(&tlv);
- return NULL;
+ tlv->length += vp->length;
}
tlv->vp_tlv = talloc_array(tlv, uint8_t, tlv->length);
if (!tlv->vp_tlv) {
- pairfree(&tlv);
+ talloc_free(tlv);
return NULL;
}
+ p = tlv->vp_tlv;
+
+ attr = 0;
+ for (vp = fr_cursor_current(&to_pack);
+ vp && vp->da->flags.is_tlv && !vp->da->flags.extended && (SUBOPTION_PARENT(vp->da->attr) == parent);
+ vp = fr_cursor_next(&to_pack)) {
+ if (SUBOPTION_ATTR(vp->da->attr) == 0) {
+ fr_strerror_printf("Invalid attribute number 0");
+ return NULL;
+ }
- ptr = tlv->vp_tlv;
- for (vp = vps; vp != NULL; vp = vp->next) {
- if (!vp->da->flags.is_tlv ||
- vp->da->flags.extended ||
- ((vp->da->attr & 0xffff00ff) != attribute)) {
- break;
+ /* Don't write out the header, were packing array options */
+ if (!vp->da->flags.array || (attr != SUBOPTION_ATTR(vp->da->attr))) {
+ attr = SUBOPTION_ATTR(vp->da->attr);
+ *p++ = attr;
+ opt_len = p++;
}
- length = fr_dhcp_vp2attr(vp, ptr + 2,
- tlv->vp_tlv + tlv->length - ptr);
- if (length > 255) {
- pairfree(&tlv);
+ length = fr_dhcp_vp2attr(p, (tlv->vp_tlv + tlv->length) - p, vp);
+ if ((length < 0) || (length > 255)) {
+ talloc_free(tlv);
return NULL;
}
+ fr_assert(opt_len);
+ *opt_len += length;
+ p += length;
+ };
+
+ return tlv;
+}
+
+/** Encode a DHCP option and any sub-options.
+ *
+ * @param out Where to write encoded DHCP attributes.
+ * @param outlen Length of out buffer.
+ * @param ctx to use for any allocated memory.
+ * @param cursor with current VP set to the option to be encoded. Will be advanced to the next option to encode.
+ * @return > 0 length of data written, < 0 error, 0 not valid option (skipping).
+ */
+ssize_t fr_dhcp_encode_option(uint8_t *out, size_t outlen, TALLOC_CTX *ctx, vp_cursor_t *cursor)
+{
+ VALUE_PAIR *vp;
+ DICT_ATTR const *previous;
+ uint8_t *opt_len, *p = out;
+ size_t freespace = outlen;
+ ssize_t len;
+
+ vp = fr_cursor_current(cursor);
+ if (!vp) return -1;
+
+ if (vp->da->vendor != DHCP_MAGIC_VENDOR) goto next; /* not a DHCP option */
+ if (vp->da->attr == 53) goto next; /* already done */
+ if ((vp->da->attr > 255) && (DHCP_BASE_ATTR(vp->da->attr) != PW_DHCP_OPTION_82)) goto next;
+
+ if (vp->da->flags.extended) {
+ next:
+ fr_strerror_printf("Attribute \"%s\" is not a DHCP option", vp->da->name);
+ fr_cursor_next(cursor);
+ return 0;
+ }
+
+ /* Write out the option number */
+ *(p++) = vp->da->attr & 0xff;
+
+ /* Pointer to the length field of the option */
+ opt_len = p++;
+
+ /* Zero out the option's length field */
+ *opt_len = 0;
+
+ /* We just consumed two bytes for the header */
+ freespace -= 2;
+
+ /* DHCP options with the same number get coalesced into a single option */
+ do {
+ VALUE_PAIR *tlv = NULL;
+
+ /* Sub option */
+ if (vp->da->flags.is_tlv) {
+ /*
+ * Coalesce TLVs into one sub-option.
+ * Cursor will be advanced to next non-TLV attribute.
+ */
+ tlv = vp = fr_dhcp_vp2suboption(ctx, cursor);
+
+ /*
+ * Skip if there's an issue coalescing the sub-options.
+ * Cursor will still have been advanced to next non-TLV attribute.
+ */
+ if (!tlv) return 0;
/*
- * Pack the attribute.
+ * If not calling fr_dhcp_vp2suboption() advance the cursor, so fr_cursor_current()
+ * returns the next attribute.
*/
- ptr[0] = (vp->da->attr & 0xff00) >> 8;
- ptr[1] = length;
+ } else {
+ fr_cursor_next(cursor);
+ }
- ptr += length + 2;
- }
+ if ((*opt_len + vp->length) > 255) {
+ fr_strerror_printf("Skipping \"%s\": Option splitting not supported "
+ "(option > 255 bytes)", vp->da->name);
+ talloc_free(tlv);
+ return 0;
+ }
- return tlv;
-}
+ len = fr_dhcp_vp2attr(p, freespace, vp);
+ talloc_free(tlv);
+ if (len < 0) {
+ /* Failed encoding option */
+ return len;
+ }
+
+ p += len;
+ *opt_len += len;
+ freespace -= len;
+ previous = vp->da;
+ } while ((vp = fr_cursor_current(cursor)) && (previous == vp->da) && vp->da->flags.array);
+
+ return p - out;
+}
int fr_dhcp_encode(RADIUS_PACKET *packet)
{
- unsigned int i, num_vps;
+ unsigned int i;
uint8_t *p;
+ vp_cursor_t cursor;
VALUE_PAIR *vp;
- uint32_t lvalue, mms;
- size_t dhcp_size, length;
+ uint32_t lvalue;
+ size_t dhcp_size;
+ ssize_t len;
#ifndef NDEBUG
- const char *name;
+ char const *name;
# ifdef WITH_UDPFROMTO
char src_ip_buf[256];
# endif
} else {
name = "?Unknown?";
}
-
+
DEBUG(
-#ifdef WITH_UDPFROMTO
+# ifdef WITH_UDPFROMTO
"Encoding %s of id %08x from %s:%d to %s:%d\n",
-#else
+# else
"Encoding %s of id %08x to %s:%d\n",
-#endif
+# endif
name, (unsigned int) packet->id,
-#ifdef WITH_UDPFROMTO
+# ifdef WITH_UDPFROMTO
inet_ntop(packet->src_ipaddr.af,
&packet->src_ipaddr.ipaddr,
src_ip_buf, sizeof(src_ip_buf)),
packet->src_port,
-#endif
+# endif
inet_ntop(packet->dst_ipaddr.af,
&packet->dst_ipaddr.ipaddr,
dst_ip_buf, sizeof(dst_ip_buf)),
p = packet->data;
+ /*
+ * @todo: Make this work again.
+ */
+#if 0
mms = DEFAULT_PACKET_SIZE; /* maximum message size */
/*
if (mms > MAX_PACKET_SIZE) mms = MAX_PACKET_SIZE;
}
-
- /*
- * RFC 3118: Authentication option.
- */
- vp = pairfind(packet->vps, 90, DHCP_MAGIC_VENDOR, TAG_ANY);
- if (vp) {
- if (vp->length < 2) {
- memset(vp->vp_octets + vp->length, 0,
- 2 - vp->length);
- vp->length = 2;
- }
-
- if (vp->length < 3) {
- struct timeval tv;
-
- gettimeofday(&tv, NULL);
- vp->vp_octets[2] = 0;
- timeval2ntp(&tv, vp->vp_octets + 3);
- vp->length = 3 + 8;
- }
-
- /*
- * Configuration token (clear-text token)
- */
- if (vp->vp_octets[0] == 0) {
- VALUE_PAIR *pass;
- vp->vp_octets[1] = 0;
-
- pass = pairfind(packet->vps, PW_CLEARTEXT_PASSWORD, DHCP_MAGIC_VENDOR, TAG_ANY);
- if (pass) {
- length = pass->length;
- if ((length + 11) > sizeof(vp->vp_octets)) {
- length -= ((length + 11) - sizeof(vp->vp_octets));
- }
- memcpy(vp->vp_octets + 11, pass->vp_strvalue,
- length);
- vp->length = length + 11;
- } else {
- vp->length = 11 + 8;
- memset(vp->vp_octets + 11, 0, 8);
- vp->length = 11 + 8;
- }
- } else { /* we don't support this type! */
- fr_strerror_printf("DHCP-Authentication %d unsupported",
- vp->vp_octets[0]);
- }
- }
+#endif
vp = pairfind(packet->vps, 256, DHCP_MAGIC_VENDOR, TAG_ANY);
if (vp) {
} else {
*p++ = 1; /* client message */
}
-
+
/* DHCP-Hardware-Type */
if ((vp = pairfind(packet->vps, 257, DHCP_MAGIC_VENDOR, TAG_ANY))) {
*p++ = vp->vp_integer & 0xFF;
/* DHCP-Client-Hardware-Address */
if ((vp = pairfind(packet->vps, 267, DHCP_MAGIC_VENDOR, TAG_ANY))) {
- if (vp->length > DHCP_CHADDR_LEN) {
- memcpy(p, vp->vp_octets, DHCP_CHADDR_LEN);
- } else {
- memcpy(p, vp->vp_octets, vp->length);
- }
+ if (vp->length == sizeof(vp->vp_ether)) {
+ memcpy(p, vp->vp_ether, vp->length);
+ } /* else ignore it */
}
p += DHCP_CHADDR_LEN;
* instead of being placed verbatim in the filename field.
*/
- /* DHCP-Boot-Filename */
- if ((vp = pairfind(packet->vps, 269, DHCP_MAGIC_VENDOR, TAG_ANY))) {
+ /* DHCP-Boot-Filename */
+ vp = pairfind(packet->vps, 269, DHCP_MAGIC_VENDOR, TAG_ANY);
+ if (vp) {
if (vp->length > DHCP_FILE_LEN) {
memcpy(p, vp->vp_strvalue, DHCP_FILE_LEN);
} else {
p = packet->data;
for (i = 0; i < 14; i++) {
+ char *q;
+
vp = pairmake(packet, NULL,
dhcp_header_names[i], NULL, T_OP_EQ);
if (!vp) {
fr_strerror_printf("Cannot decode packet due to internal error: %s", buffer);
return -1;
}
-
+
switch (vp->da->type) {
case PW_TYPE_BYTE:
- vp->vp_integer = p[0];
- vp->length = 1;
+ vp->vp_byte = p[0];
break;
case PW_TYPE_SHORT:
- vp->vp_integer = (p[0] << 8) | p[1];
- vp->length = 2;
+ vp->vp_short = (p[0] << 8) | p[1];
break;
case PW_TYPE_INTEGER:
memcpy(&vp->vp_integer, p, 4);
vp->vp_integer = ntohl(vp->vp_integer);
- vp->length = 4;
break;
- case PW_TYPE_IPADDR:
+ case PW_TYPE_IPV4_ADDR:
memcpy(&vp->vp_ipaddr, p, 4);
- vp->length = 4;
break;
case PW_TYPE_STRING:
- memcpy(vp->vp_strvalue, p, dhcp_header_sizes[i]);
- vp->vp_strvalue[dhcp_header_sizes[i]] = '\0';
+ vp->vp_strvalue = q = talloc_array(vp, char, dhcp_header_sizes[i] + 1);
+ vp->type = VT_DATA;
+ memcpy(q, p, dhcp_header_sizes[i]);
+ q[dhcp_header_sizes[i]] = '\0';
vp->length = strlen(vp->vp_strvalue);
break;
case PW_TYPE_OCTETS: /* only for Client HW Address */
- memcpy(vp->vp_octets, p, packet->data[2]);
- vp->length = packet->data[2];
+ pairmemcpy(vp, p, packet->data[2]);
break;
case PW_TYPE_ETHERNET: /* only for Client HW Address */
memcpy(vp->vp_ether, p, sizeof(vp->vp_ether));
- vp->length = sizeof(vp->vp_ether);
break;
default:
p = pp;
}
- /*
- * Before packing the attributes, re-order them so that
- * the array ones are all contiguous. This simplifies
- * the later code.
- */
- num_vps = 0;
- for (vp = packet->vps; vp != NULL; vp = vp->next) {
- num_vps++;
- }
- if (num_vps > 1) {
- VALUE_PAIR **array, **last;
-
- array = talloc_array(packet, VALUE_PAIR*, num_vps);
-
- i = 0;
- for (vp = packet->vps; vp != NULL; vp = vp->next) {
- array[i++] = vp;
- }
-
- /*
- * Sort the attributes.
- */
- qsort(array, (size_t) num_vps, sizeof(VALUE_PAIR *),
- attr_cmp);
-
- last = &packet->vps;
- for (i = 0; i < num_vps; i++) {
- *last = array[i];
- array[i]->next = NULL;
- last = &(array[i]->next);
- }
- talloc_free(array);
- }
-
p[0] = 0x35; /* DHCP-Message-Type */
p[1] = 1;
p[2] = packet->code - PW_DHCP_OFFSET;
p += 3;
+
/*
- * Pack in the attributes.
+ * Pre-sort attributes into contiguous blocks so that fr_dhcp_encode_option
+ * operates correctly. This changes the order of the list, but never mind...
*/
- vp = packet->vps;
- while (vp) {
- unsigned int num_entries = 1;
- VALUE_PAIR *same;
- uint8_t *plength;
-
- if (vp->da->vendor != DHCP_MAGIC_VENDOR) goto next;
- if (vp->da->attr == 53) goto next; /* already done */
- if ((vp->da->attr > 255) &&
- (DHCP_BASE_ATTR(vp->da->attr) != PW_DHCP_OPTION_82)) goto next;
-
- debug_pair(vp);
- if (vp->da->flags.extended) goto next;
-
- length = vp->length;
-
- for (same = vp->next; same != NULL; same = same->next) {
- if (same->da->attr != vp->da->attr) break;
- num_entries++;
- }
-
- /*
- * For client-identifier
- * @fixme What's this meant to be doing?!
- */
-#if 0
- if ((vp->da->type == PW_TYPE_ETHERNET) &&
- (vp->length == 6) &&
- (num_entries == 1)) {
- vp->da->type = PW_TYPE_OCTETS;
- memmove(vp->vp_octets + 1, vp->vp_octets, 6);
- vp->vp_octets[0] = 1;
- }
-#endif
- *(p++) = vp->da->attr & 0xff;
- plength = p;
- *(p++) = 0; /* header isn't included in attr length */
-
- for (i = 0; i < num_entries; i++) {
- if (i != 0) debug_pair(vp);
-
- if (vp->da->flags.is_tlv) {
- VALUE_PAIR *tlv;
-
- /*
- * Should NOT have been encoded yet!
- */
- tlv = fr_dhcp_vp2suboption(packet, vp);
-
- /*
- * Ignore it if there's an issue
- * encoding it.
- */
- if (!tlv) goto next;
-
- tlv->next = vp->next;
- vp->next = tlv;
- vp = tlv;
- }
-
- length = fr_dhcp_vp2attr(vp, p, 0);
-
- /*
- * This will never happen due to FreeRADIUS
- * limitations: sizeof(vp->vp_octets) < 255
- */
- if (length > 255) {
- fr_strerror_printf("WARNING Ignoring too long attribute %s!", vp->da->name);
- break;
- }
-
- /*
- * More than one attribute of the same type
- * in a row: they are packed together
- * into the same TLV. If we overflow,
- * go bananas!
- */
- if ((*plength + length) > 255) {
- fr_strerror_printf("WARNING Ignoring too long attribute %s!", vp->da->name);
- break;
- }
-
- *plength += length;
- p += length;
-
- if (vp->next &&
- (vp->next->da->attr == vp->da->attr))
- vp = vp->next;
- } /* loop over num_entries */
+ pairsort(&packet->vps, fr_dhcp_attr_cmp);
+ fr_cursor_init(&cursor, &packet->vps);
- next:
- vp = vp->next;
- }
+ /*
+ * Each call to fr_dhcp_encode_option will encode one complete DHCP option,
+ * and sub options.
+ */
+ while ((vp = fr_cursor_current(&cursor))) {
+ len = fr_dhcp_encode_option(p, packet->data_len - (p - packet->data), packet, &cursor);
+ if (len < 0) break;
+ if (len > 0) debug_pair(vp);
+ p += len;
+ };
p[0] = 0xff; /* end of option option */
p[1] = 0x00;
}
if ((fr_debug_flag > 2) && fr_log_fp) {
+ fprintf(fr_log_fp, "DHCP Sending %zu bytes\n", packet->data_len);
for (i = 0; i < packet->data_len; i++) {
if ((i & 0x0f) == 0x00) fprintf(fr_log_fp, "%d: ", i);
fprintf(fr_log_fp, "%02x ", packet->data[i]);
}
#ifdef SIOCSARP
-int fr_dhcp_add_arp_entry(int fd, const char *interface,
+int fr_dhcp_add_arp_entry(int fd, char const *interface,
VALUE_PAIR *macaddr, VALUE_PAIR *ip)
{
struct sockaddr_in *sin;
struct arpreq req;
- if (macaddr->length > sizeof (req.arp_ha.sa_data)) {
- fr_strerror_printf("ERROR: DHCP only supports up to %zu octets "
- "for Client Hardware Address "
- "(got %zu octets)\n",
- sizeof(req.arp_ha.sa_data),
- macaddr->length);
+ if (!interface) {
+ fr_strerror_printf("No interface specified. Cannot update ARP table");
+ return -1;
+ }
+
+ if (!fr_assert(macaddr) ||
+ !fr_assert((macaddr->da->type == PW_TYPE_ETHERNET) || (macaddr->da->type == PW_TYPE_OCTETS))) {
+ fr_strerror_printf("Wrong VP type (%s) for chaddr",
+ fr_int2str(dict_attr_types, macaddr->da->type, "<invalid>"));
+ return -1;
+ }
+
+ if (macaddr->length > sizeof(req.arp_ha.sa_data)) {
+ fr_strerror_printf("arp sa_data field too small (%zu octets) to contain chaddr (%zu octets)",
+ sizeof(req.arp_ha.sa_data), macaddr->length);
return -1;
}
sin = (struct sockaddr_in *) &req.arp_pa;
sin->sin_family = AF_INET;
sin->sin_addr.s_addr = ip->vp_ipaddr;
+
strlcpy(req.arp_dev, interface, sizeof(req.arp_dev));
- memcpy(&req.arp_ha.sa_data, macaddr->vp_octets, macaddr->length);
+
+ if (macaddr->da->type == PW_TYPE_ETHERNET) {
+ memcpy(&req.arp_ha.sa_data, &macaddr->vp_ether, sizeof(macaddr->vp_ether));
+ } else {
+ memcpy(&req.arp_ha.sa_data, macaddr->vp_octets, macaddr->length);
+ }
req.arp_flags = ATF_COM;
if (ioctl(fd, SIOCSARP, &req) < 0) {
- fr_strerror_printf("DHCP: Failed to add entry in ARP cache: %s (%d)",
- strerror(errno), errno);
+ fr_strerror_printf("Failed to add entry in ARP cache: %s (%d)", fr_syserror(errno), errno);
return -1;
}
return 0;
}
#else
-int fr_dhcp_add_arp_entry(UNUSED int fd, UNUSED const char *interface,
+int fr_dhcp_add_arp_entry(UNUSED int fd, UNUSED char const *interface,
UNUSED VALUE_PAIR *macaddr, UNUSED VALUE_PAIR *ip)
{
fr_strerror_printf("Adding ARP entry is unsupported on this system");