Code cleanup

This commit is contained in:
angt
2016-01-04 20:07:11 +01:00
parent 8fa2322314
commit bfd81e9666
2 changed files with 52 additions and 73 deletions

View File

@@ -2,56 +2,38 @@
#include <stdint.h> #include <stdint.h>
struct ip_common {
uint8_t version;
uint8_t proto;
uint8_t hdr_size;
uint16_t size;
};
_pure_ _pure_
static inline int ip_get_version (const uint8_t *data, size_t size) static inline uint8_t ip_get_version (const uint8_t *data, size_t size)
{ {
if (size<20) // XXX if (size<20)
return -1; // XXX return 0;
return data[0]>>4; return data[0]>>4;
} }
_pure_ static inline int ip_get_common (struct ip_common *ic, const uint8_t *data, size_t size)
static inline ssize_t ip_get_size (const int ip_version, const uint8_t *data, size_t size)
{ {
switch (ip_version) { ic->version = ip_get_version(data, size);
switch (ic->version) {
case 4: case 4:
return ((data[2]<<8)|data[3]); ic->proto = data[9];
ic->hdr_size = (data[0]&0xF)<<2;
ic->size = ((data[2]<<8)|data[3]);
return 0;
case 6: case 6:
return ((data[4]<<8)|data[5])+40; ic->proto = data[6];
case -1: ic->hdr_size = 40;
return -1; ic->size = ((data[4]<<8)|data[5])+40;
return 0;
} }
return 0; return -1;
}
_pure_
static inline ssize_t ip_get_proto (const int ip_version, const uint8_t *data, size_t size)
{
switch (ip_version) {
case 4:
return data[9];
case 6:
return data[6];
case -1:
return -1;
}
return 0;
}
_pure_
static inline ssize_t ip_get_hdr_size (const int ip_version, const uint8_t *data, size_t size)
{
switch (ip_version) {
case 4:
return (data[0]&0xF)<<2;
case 6:
return 40;
case -1:
return -1;
}
return 0;
} }

View File

@@ -533,20 +533,17 @@ static inline uint16_t sum16_final (uint32_t sum)
return ~(sum+(sum>>16)); return ~(sum+(sum>>16));
} }
static void gt_print_hdr (const int ip_version, uint8_t *data, size_t ip_size) static void gt_print_hdr (struct ip_common *ic, uint8_t *data)
{ {
const ssize_t ip_proto = ip_get_proto(ip_version, data, ip_size); if (!ic->hdr_size)
const ssize_t ip_hdr_size = ip_get_hdr_size(ip_version, data, ip_size);
if (ip_proto<0 || ip_hdr_size<=0)
return; return;
uint32_t sum = (size_t)ip_proto+ip_size-(size_t)ip_hdr_size; uint32_t sum = ic->proto+ic->size-ic->hdr_size;
char ip_src[INET6_ADDRSTRLEN]; char ip_src[INET6_ADDRSTRLEN];
char ip_dst[INET6_ADDRSTRLEN]; char ip_dst[INET6_ADDRSTRLEN];
switch (ip_version) { switch (ic->version) {
case 4: case 4:
inet_ntop(AF_INET, &data[12], ip_src, sizeof(ip_src)); inet_ntop(AF_INET, &data[12], ip_src, sizeof(ip_src));
inet_ntop(AF_INET, &data[16], ip_dst, sizeof(ip_dst)); inet_ntop(AF_INET, &data[16], ip_dst, sizeof(ip_dst));
@@ -559,16 +556,18 @@ static void gt_print_hdr (const int ip_version, uint8_t *data, size_t ip_size)
break; break;
} }
if (ip_proto==IPPROTO_TCP) { uint8_t *const packet = &data[ic->hdr_size];
if (ic->proto==IPPROTO_TCP) {
struct tcphdr tcp; struct tcphdr tcp;
byte_cpy(&tcp, &data[ip_hdr_size], sizeof(tcp)); byte_cpy(&tcp, packet, sizeof(tcp));
uint16_t tcp_sum = ntohs(tcp.th_sum); uint16_t tcp_sum = ntohs(tcp.th_sum);
tcp.th_sum = 0; tcp.th_sum = 0;
sum = sum16(sum, (uint8_t *)&tcp, sizeof(tcp)); sum = sum16(sum, (uint8_t *)&tcp, sizeof(tcp));
sum = sum16(sum, &data[ip_hdr_size+sizeof(tcp)], ip_size-ip_hdr_size-sizeof(tcp)); sum = sum16(sum, &packet[sizeof(tcp)], ic->size-ic->hdr_size-sizeof(tcp));
uint16_t computed_sum = sum16_final(sum); uint16_t computed_sum = sum16_final(sum);
tcp.th_sport = ntohs(tcp.th_sport); tcp.th_sport = ntohs(tcp.th_sport);
@@ -577,9 +576,9 @@ static void gt_print_hdr (const int ip_version, uint8_t *data, size_t ip_size)
tcp.th_ack = ntohl(tcp.th_ack); tcp.th_ack = ntohl(tcp.th_ack);
tcp.th_win = ntohs(tcp.th_win); tcp.th_win = ntohs(tcp.th_win);
gt_print("proto:%zi\tsrc:%s.%u\tdst:%s.%u\tseq:%u\tack:%u\twin:%u\tsize:%zu\tflags:%c%c%c%c%c%c\tsum:%i\n", gt_print("proto:%hhu\tsrc:%s.%u\tdst:%s.%u\tseq:%u\tack:%u\twin:%u\tsize:%u\tflags:%c%c%c%c%c%c\tsum:%i\n",
ip_proto, ip_src, tcp.th_sport, ip_dst, tcp.th_dport, ic->proto, ip_src, tcp.th_sport, ip_dst, tcp.th_dport,
tcp.th_seq, tcp.th_ack, tcp.th_win, ip_size-ip_hdr_size+tcp.th_off*4, tcp.th_seq, tcp.th_ack, tcp.th_win, ic->size-ic->hdr_size-tcp.th_off*4,
(tcp.th_flags&TH_FIN) ?'F':'.', (tcp.th_flags&TH_FIN) ?'F':'.',
(tcp.th_flags&TH_SYN) ?'S':'.', (tcp.th_flags&TH_SYN) ?'S':'.',
(tcp.th_flags&TH_RST) ?'R':'.', (tcp.th_flags&TH_RST) ?'R':'.',
@@ -588,20 +587,20 @@ static void gt_print_hdr (const int ip_version, uint8_t *data, size_t ip_size)
(tcp.th_flags&TH_URG) ?'U':'.', (tcp.th_flags&TH_URG) ?'U':'.',
(computed_sum==tcp_sum)); (computed_sum==tcp_sum));
} else if (ip_proto==IPPROTO_UDP) { } else if (ic->proto==IPPROTO_UDP) {
struct udphdr udp; struct udphdr udp;
byte_cpy(&udp, &data[ip_hdr_size], sizeof(udp)); byte_cpy(&udp, packet, sizeof(udp));
udp.uh_sport = ntohs(udp.uh_sport); udp.uh_sport = ntohs(udp.uh_sport);
udp.uh_dport = ntohs(udp.uh_dport); udp.uh_dport = ntohs(udp.uh_dport);
udp.uh_ulen = ntohs(udp.uh_ulen); udp.uh_ulen = ntohs(udp.uh_ulen);
gt_print("proto:%zi\tsrc:%s.%u\tdst:%s.%u\tsize:%u\n", gt_print("proto:%hhu\tsrc:%s.%u\tdst:%s.%u\tsize:%u\n",
ip_proto, ip_src, udp.uh_sport, ip_dst, udp.uh_dport, udp.uh_ulen-8); ic->proto, ip_src, udp.uh_sport, ip_dst, udp.uh_dport, udp.uh_ulen-8);
} else { } else {
gt_print("proto:%zi\tsrc:%s\tdst:%s\tsize:%zu\n", gt_print("proto:%hhu\tsrc:%s\tdst:%s\tsize:%hu\n",
ip_proto, ip_src, ip_dst, ip_size); ic->proto, ip_src, ip_dst, ic->size);
} }
} }
@@ -1055,13 +1054,12 @@ int main (int argc, char **argv)
break; break;
} }
const int ip_version = ip_get_version(data, GT_MTU_MAX); struct ip_common ic;
const ssize_t ip_size = ip_get_size(ip_version, data, GT_MTU_MAX);
if _0_(ip_size<=0) if (ip_get_common(&ic, data, GT_MTU_MAX))
continue; continue;
if _0_(ip_size!=r) { if _0_(ic.size!=r) {
char tmp[2*GT_MTU_MAX+1]; char tmp[2*GT_MTU_MAX+1];
gt_tohex(tmp, sizeof(tmp), data, r); gt_tohex(tmp, sizeof(tmp), data, r);
gt_log("%s: DUMP %zi %s\n", sockname, r, tmp); gt_log("%s: DUMP %zi %s\n", sockname, r, tmp);
@@ -1069,7 +1067,7 @@ int main (int argc, char **argv)
} }
if _0_(debug) if _0_(debug)
gt_print_hdr(ip_version, data, ip_size); gt_print_hdr(&ic, data);
blks[blk_write++].size = r; blks[blk_write++].size = r;
blk_count++; blk_count++;
@@ -1145,22 +1143,21 @@ int main (int argc, char **argv)
size_t size = buffer_read_size(&tun.write); size_t size = buffer_read_size(&tun.write);
const int ip_version = ip_get_version(tun.write.read, size); if (!size)
ssize_t ip_size = ip_get_size(ip_version, tun.write.read, size); break;
if _0_(!ip_size) { struct ip_common ic;
if (ip_get_common(&ic, tun.write.read, size) || ic.size>size) {
gt_log("%s: bad packet!\n", sockname); gt_log("%s: bad packet!\n", sockname);
goto restart; goto restart;
} }
if (ip_size<0 || (size_t)ip_size>size) ssize_t r = tun_write(tun.fd, tun.write.read, ic.size);
break;
ssize_t r = tun_write(tun.fd, tun.write.read, ip_size);
if (r>0) { if (r>0) {
if _0_(debug) if _0_(debug)
gt_print_hdr(ip_version, tun.write.read, ip_size); gt_print_hdr(&ic, tun.write.read);
tun.write.read += r; tun.write.read += r;
} else { } else {
gt_close |= !r; gt_close |= !r;