貌似发现一个FreeBSD内核的小bug

代码中只判断了IP头是否在mbuf的空间内，而实际上ipv4和ipv6头里的IP_TOS有可能在mbuf之外。不过二层头一般应该不会大到把ip头挤到mbuf最后一个字节，从而使ip_tos在另一个mbuf内。
/*
* read and write diffserv field in IPv4 or IPv6 header
*/
u_int8_t
read_dsfield(m, pktattr)
    struct mbuf *m;
    struct altq_pktattr *pktattr;
{
    struct mbuf *m0;
    u_int8_t ds_field = 0;
    if (pktattr == NULL ||
        (pktattr->pattr_af != AF_INET && pktattr->pattr_af != AF_INET6))
        return ((u_int8_t)0);
    /* verify that pattr_hdr is within the mbuf data */
    for (m0 = m; m0 != NULL; m0 = m0->m_next)
        if ((pktattr->pattr_hdr >= m0->m_data) &&
            (pktattr->pattr_hdr  m0->m_data + m0->m_len))
            break;
    if (m0 == NULL) {
        /* ick, pattr_hdr is stale */
        pktattr->pattr_af = AF_UNSPEC;
#ifdef ALTQ_DEBUG
        printf("read_dsfield: can't locate header!\n");
#endif
        return ((u_int8_t)0);
    }
    if (pktattr->pattr_af == AF_INET) {
        struct ip *ip = (struct ip *)pktattr->pattr_hdr;
        if (ip->ip_v != 4)
            return ((u_int8_t)0);    /* version mismatch! */
        ds_field = ip->ip_tos;
    }
#ifdef INET6
    else if (pktattr->pattr_af == AF_INET6) {
        struct ip6_hdr *ip6 = (struct ip6_hdr *)pktattr->pattr_hdr;
        u_int32_t flowlabel;
        flowlabel = ntohl(ip6->ip6_flow);
        if ((flowlabel >> 28) != 6)
            return ((u_int8_t)0);    /* version mismatch! */
        ds_field = (flowlabel >> 20) & 0xff;
    }
#endif
    return (ds_field);
}
void
write_dsfield(m, pktattr, dsfield)
    struct mbuf *m;
    struct altq_pktattr *pktattr;
    u_int8_t dsfield;
{
    struct mbuf *m0;
    if (pktattr == NULL ||
        (pktattr->pattr_af != AF_INET && pktattr->pattr_af != AF_INET6))
        return;

标签：

版权申明：本站文章部分自网络，如有侵权，请联系：west999com@outlook.com
特别注意：本站所有转载文章言论不代表本站观点，本站所提供的摄影照片，插画，设计作品，如需使用，请与原作者联系，版权归原作者所有