From 528c8b904a1da81934ac8d8d7dcbf97f220a7228 Mon Sep 17 00:00:00 2001 From: Ralf Baechle Date: Tue, 23 Jan 2007 13:40:01 +0000 Subject: Fix warnings about // style comment. And remove some of the most pointless comments. --- ax25/rxecho.c | 54 ++++++++++++++++++++++++++++++------------------------ 1 file changed, 30 insertions(+), 24 deletions(-) (limited to 'ax25/rxecho.c') diff --git a/ax25/rxecho.c b/ax25/rxecho.c index 361454b..eb46810 100644 --- a/ax25/rxecho.c +++ b/ax25/rxecho.c @@ -67,16 +67,18 @@ #include -// dl9sau: -// uncomment this if you have problems with sockaddr_pkt. -// sockaddr_pkt is the right way for type SOCK_PACKET on family AF_INET sockets. -// especially because the "sockaddr" on recvfrom is truncated (internaly -// it's sockaddr_pkt) -- and because we use this sockaddr for retransmitting, -// it's really better to use the sockaddr_spkt. -// default is to use SOCKADDR_SPKT +/* + * dl9sau: + * uncomment this if you have problems with sockaddr_pkt. + * sockaddr_pkt is the right way for type SOCK_PACKET on family AF_INET sockets. + * especially because the "sockaddr" on recvfrom is truncated (internaly + * it's sockaddr_pkt) -- and because we use this sockaddr for retransmitting, + * it's really better to use the sockaddr_spkt. + * default is to use SOCKADDR_SPKT + */ #define USE_SOCKADDR_SPKT 1 -//dl9sau: since linux 2.2.x, SOCK_PACKET is obsolete +/* dl9sau: since linux 2.2.x, SOCK_PACKET is obsolete */ #define USE_SOCKADDR_SLL 1 #ifdef USE_SOCKADDR_SLL @@ -428,7 +430,7 @@ int main(int argc, char **argv) char *p_name = (i ? p->to : p->from); int *p_idx = (i ? &p->to_idx : &p->from_idx); - // already set? + /* already set? */ if (*p_idx >= 0) continue; strncpy(ifr.ifr_name, p_name, sizeof(ifr.ifr_name)-1); @@ -473,9 +475,11 @@ int main(int argc, char **argv) #ifdef USE_SOCKADDR_SLL from_idx = sll.sll_ifindex; #else - // dl9sau: save the names of the iface the frame came from; - // we'll overwrite psa->sa_data it for sendto() and need the - // name again when multiplexing to more than one iface + /* + * dl9sau: save the names of the iface the frame came from; + * we'll overwrite psa->sa_data it for sendto() and need the + * name again when multiplexing to more than one iface + */ strncpy(from_dev_name, psa->sa_data, sizeof(from_dev_name)-1); from_dev_name[sizeof(from_dev_name)-1] = 0; #endif @@ -488,18 +492,20 @@ int main(int argc, char **argv) if ((strcmp(p->from, from_dev_name) == 0) && (check_calls(p, buf, size) == 0)) { strcpy(psa->sa_data, p->to); #endif - // cave: alen (set by recvfrom()) may > salen - // which means it may point beyound of the - // end of the struct. thats why we use the - // correct len (sa_len). this fixes a bug - // where skpt->protocol on sockaddr structs - // pointed behind the struct, leading to - // funny protocol IDs. - // btw, the linux kernel internaly always - // maps to sockaddr to sockaddr_pkt - // on sockets of type SOCK_PACKET on family - // AF_INET. sockaddr_pkt is len 18, sockaddr - // is 16. + /* + * cave: alen (set by recvfrom()) may > salen + * which means it may point beyound of the + * end of the struct. thats why we use the + * correct len (sa_len). this fixes a bug + * where skpt->protocol on sockaddr structs + * pointed behind the struct, leading to + * funny protocol IDs. + * btw, the linux kernel internaly always + * maps to sockaddr to sockaddr_pkt + * on sockets of type SOCK_PACKET on family + * AF_INET. sockaddr_pkt is len 18, sockaddr + * is 16. + */ if (sendto(s, buf, size, 0, psa, sa_len) == -1) { if (logging) { syslog(LOG_ERR, "sendto: %m"); -- cgit v1.2.3