There is the only way to reach ip_options compile with opt != NULL:
ip_options_get_finish
    opt->is_data = 1;
    ip_options_compile(opt, NULL)
So, checking for is_data inside opt != NULL branch is not needed.
Signed-off-by: Denis V. Lunev <den@openvz.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
                optptr = iph + sizeof(struct iphdr);
                opt->is_data = 0;
        } else {
-               optptr = opt->is_data ? opt->__data :
-                                       (unsigned char *)&(ip_hdr(skb)[1]);
+               optptr = opt->__data;
                iph = optptr - sizeof(struct iphdr);
        }