]> git.ipfire.org Git - thirdparty/freeradius-server.git/commitdiff
Always write the header for the first attr
authorAlan T. DeKok <aland@freeradius.org>
Mon, 15 Jun 2015 18:30:22 +0000 (14:30 -0400)
committerAlan T. DeKok <aland@freeradius.org>
Mon, 15 Jun 2015 18:30:22 +0000 (14:30 -0400)
src/modules/proto_dhcp/dhcp.c

index b4eafda319af6d023dd26b7eb94e94cf267182ae..48fc3a28d1babecdb36b662b41fedcb85b111b05 100644 (file)
@@ -1266,7 +1266,7 @@ static ssize_t fr_dhcp_vp2data_tlv(uint8_t *out, ssize_t outlen, vp_cursor_t *cu
             vp && vp->da->flags.is_tlv && (SUBOPTION_PARENT(vp->da->attr) == parent);
             vp = fr_cursor_next(cursor)) {
                /* Don't write out the header, were packing array options */
-               if (!vp->da->flags.array || (attr != SUBOPTION_ATTR(vp->da->attr))) {
+               if (!opt_len || !vp->da->flags.array || (attr != SUBOPTION_ATTR(vp->da->attr))) {
                        attr = SUBOPTION_ATTR(vp->da->attr);
                        *p++ = attr;
                        opt_len = p++;