From 45a5d80c49236afc02ad5f45264be185300da8eb Mon Sep 17 00:00:00 2001
From: Erik Ekman <erik@kryo.se>
Date: Mon, 6 Feb 2012 20:28:42 +0100
Subject: [PATCH] Adjust indentation

---
 src/client.c  | 294 +++++++++++++++++++++++---------------------------
 src/dns.c     |  16 ++-
 src/iodined.c |  40 ++++---
 3 files changed, 163 insertions(+), 187 deletions(-)

diff --git a/src/client.c b/src/client.c
index 9f5dfe7..8bb972f 100644
--- a/src/client.c
+++ b/src/client.c
@@ -1458,32 +1458,30 @@ handshake_version(int dns_fd, int *seed)
 
 		read = handshake_waitdns(dns_fd, in, sizeof(in), 'v', 'V', i+1);
 
-		/*XXX START adjust indent 1 tab back*/
-			if (read >= 9) {
-				payload =  (((in[4] & 0xff) << 24) |
-						((in[5] & 0xff) << 16) |
-						((in[6] & 0xff) << 8) |
-						((in[7] & 0xff)));
+		if (read >= 9) {
+			payload =  (((in[4] & 0xff) << 24) |
+					((in[5] & 0xff) << 16) |
+					((in[6] & 0xff) << 8) |
+					((in[7] & 0xff)));
 
-				if (strncmp("VACK", in, 4) == 0) {
-					*seed = payload;
-					userid = in[8];
-					userid_char = hex[userid & 15];
-					userid_char2 = hex2[userid & 15];
+			if (strncmp("VACK", in, 4) == 0) {
+				*seed = payload;
+				userid = in[8];
+				userid_char = hex[userid & 15];
+				userid_char2 = hex2[userid & 15];
 
-					fprintf(stderr, "Version ok, both using protocol v 0x%08x. You are user #%d\n", VERSION, userid);
-					return 0;
-				} else if (strncmp("VNAK", in, 4) == 0) {
-					warnx("You use protocol v 0x%08x, server uses v 0x%08x. Giving up", 
-							VERSION, payload);
-					return 1;
-				} else if (strncmp("VFUL", in, 4) == 0) {
-					warnx("Server full, all %d slots are taken. Try again later", payload);
-					return 1;
-				}
-			} else if (read > 0)
-				warnx("did not receive proper login challenge");
-		/*XXX END adjust indent 1 tab back*/
+				fprintf(stderr, "Version ok, both using protocol v 0x%08x. You are user #%d\n", VERSION, userid);
+				return 0;
+			} else if (strncmp("VNAK", in, 4) == 0) {
+				warnx("You use protocol v 0x%08x, server uses v 0x%08x. Giving up", 
+						VERSION, payload);
+				return 1;
+			} else if (strncmp("VFUL", in, 4) == 0) {
+				warnx("Server full, all %d slots are taken. Try again later", payload);
+				return 1;
+			}
+		} else if (read > 0)
+			warnx("did not receive proper login challenge");
 		
 		fprintf(stderr, "Retrying version check...\n");
 	}
@@ -1510,30 +1508,28 @@ handshake_login(int dns_fd, int seed)
 
 		read = handshake_waitdns(dns_fd, in, sizeof(in), 'l', 'L', i+1);
 
-		/*XXX START adjust indent 1 tab back*/
-			if (read > 0) {
-				int netmask;
-				if (strncmp("LNAK", in, 4) == 0) {
-					fprintf(stderr, "Bad password\n");
-					return 1;
-				} else if (sscanf(in, "%64[^-]-%64[^-]-%d-%d", 
-					server, client, &mtu, &netmask) == 4) {
-					
-					server[64] = 0;
-					client[64] = 0;
-					if (tun_setip(client, server, netmask) == 0 && 
-						tun_setmtu(mtu) == 0) {
+		if (read > 0) {
+			int netmask;
+			if (strncmp("LNAK", in, 4) == 0) {
+				fprintf(stderr, "Bad password\n");
+				return 1;
+			} else if (sscanf(in, "%64[^-]-%64[^-]-%d-%d",
+				server, client, &mtu, &netmask) == 4) {
 
-						fprintf(stderr, "Server tunnel IP is %s\n", server);
-						return 0;
-					} else {
-						errx(4, "Failed to set IP and MTU");
-					}
+				server[64] = 0;
+				client[64] = 0;
+				if (tun_setip(client, server, netmask) == 0 &&
+					tun_setmtu(mtu) == 0) {
+
+					fprintf(stderr, "Server tunnel IP is %s\n", server);
+					return 0;
 				} else {
-					fprintf(stderr, "Received bad handshake\n");
+					errx(4, "Failed to set IP and MTU");
 				}
+			} else {
+				fprintf(stderr, "Received bad handshake\n");
 			}
-		/*XXX END adjust indent 1 tab back*/
+		}
 
 		fprintf(stderr, "Retrying login...\n");
 	}
@@ -1560,20 +1556,18 @@ handshake_raw_udp(int dns_fd, int seed)
 
 		len = handshake_waitdns(dns_fd, in, sizeof(in), 'i', 'I', i+1);
 
-		/*XXX START adjust indent 1 tab back*/
-			if (len == 5 && in[0] == 'I') {
-				/* Received IP address */
-				remoteaddr = (in[1] & 0xff);
-				remoteaddr <<= 8;
-				remoteaddr |= (in[2] & 0xff);
-				remoteaddr <<= 8;
-				remoteaddr |= (in[3] & 0xff);
-				remoteaddr <<= 8;
-				remoteaddr |= (in[4] & 0xff);
-				server.s_addr = ntohl(remoteaddr);
-				break;
-			}
-		/*XXX END adjust indent 1 tab back*/
+		if (len == 5 && in[0] == 'I') {
+			/* Received IP address */
+			remoteaddr = (in[1] & 0xff);
+			remoteaddr <<= 8;
+			remoteaddr |= (in[2] & 0xff);
+			remoteaddr <<= 8;
+			remoteaddr |= (in[3] & 0xff);
+			remoteaddr <<= 8;
+			remoteaddr |= (in[4] & 0xff);
+			server.s_addr = ntohl(remoteaddr);
+			break;
+		}
 
 		fprintf(stderr, ".");
 		fflush(stderr);
@@ -2098,24 +2092,22 @@ handshake_switch_codec(int dns_fd, int bits)
 		
 		read = handshake_waitdns(dns_fd, in, sizeof(in), 's', 'S', i+1);
 
-		/*XXX START adjust indent 1 tab back*/			
-			if (read > 0) {
-				if (strncmp("BADLEN", in, 6) == 0) {
-					fprintf(stderr, "Server got bad message length. ");
-					goto codec_revert;
-				} else if (strncmp("BADIP", in, 5) == 0) {
-					fprintf(stderr, "Server rejected sender IP address. ");
-					goto codec_revert;
-				} else if (strncmp("BADCODEC", in, 8) == 0) {
-					fprintf(stderr, "Server rejected the selected codec. ");
-					goto codec_revert;
-				}
-				in[read] = 0; /* zero terminate */
-				fprintf(stderr, "Server switched upstream to codec %s\n", in);
-				dataenc = tempenc;
-				return;
+		if (read > 0) {
+			if (strncmp("BADLEN", in, 6) == 0) {
+				fprintf(stderr, "Server got bad message length. ");
+				goto codec_revert;
+			} else if (strncmp("BADIP", in, 5) == 0) {
+				fprintf(stderr, "Server rejected sender IP address. ");
+				goto codec_revert;
+			} else if (strncmp("BADCODEC", in, 8) == 0) {
+				fprintf(stderr, "Server rejected the selected codec. ");
+				goto codec_revert;
 			}
-		/*XXX END adjust indent 1 tab back*/
+			in[read] = 0; /* zero terminate */
+			fprintf(stderr, "Server switched upstream to codec %s\n", in);
+			dataenc = tempenc;
+			return;
+		}
 
 		fprintf(stderr, "Retrying codec switch...\n");
 	}
@@ -2153,23 +2145,21 @@ handshake_switch_downenc(int dns_fd)
 
 		read = handshake_waitdns(dns_fd, in, sizeof(in), 'o', 'O', i+1);
 
-		/*XXX START adjust indent 1 tab back*/
-			if (read > 0) {
-				if (strncmp("BADLEN", in, 6) == 0) {
-					fprintf(stderr, "Server got bad message length. ");
-					goto codec_revert;
-				} else if (strncmp("BADIP", in, 5) == 0) {
-					fprintf(stderr, "Server rejected sender IP address. ");
-					goto codec_revert;
-				} else if (strncmp("BADCODEC", in, 8) == 0) {
-					fprintf(stderr, "Server rejected the selected codec. ");
-					goto codec_revert;
-				}
-				in[read] = 0; /* zero terminate */
-				fprintf(stderr, "Server switched downstream to codec %s\n", in);
-				return;
+		if (read > 0) {
+			if (strncmp("BADLEN", in, 6) == 0) {
+				fprintf(stderr, "Server got bad message length. ");
+				goto codec_revert;
+			} else if (strncmp("BADIP", in, 5) == 0) {
+				fprintf(stderr, "Server rejected sender IP address. ");
+				goto codec_revert;
+			} else if (strncmp("BADCODEC", in, 8) == 0) {
+				fprintf(stderr, "Server rejected the selected codec. ");
+				goto codec_revert;
 			}
-		/*XXX END adjust indent 1 tab back*/
+			in[read] = 0; /* zero terminate */
+			fprintf(stderr, "Server switched downstream to codec %s\n", in);
+			return;
+		}
 
 		fprintf(stderr, "Retrying codec switch...\n");
 	}
@@ -2196,24 +2186,22 @@ handshake_try_lazy(int dns_fd)
 
 		read = handshake_waitdns(dns_fd, in, sizeof(in), 'o', 'O', i+1);
 
-		/*XXX START adjust indent 1 tab back*/
-			if (read > 0) {
-				if (strncmp("BADLEN", in, 6) == 0) {
-					fprintf(stderr, "Server got bad message length. ");
-					goto codec_revert;
-				} else if (strncmp("BADIP", in, 5) == 0) {
-					fprintf(stderr, "Server rejected sender IP address. ");
-					goto codec_revert;
-				} else if (strncmp("BADCODEC", in, 8) == 0) {
-					fprintf(stderr, "Server rejected lazy mode. ");
-					goto codec_revert;
-				} else if (strncmp("Lazy", in, 4) == 0) {
-					fprintf(stderr, "Server switched to lazy mode\n");
-					lazymode = 1;
-					return;
-				}
+		if (read > 0) {
+			if (strncmp("BADLEN", in, 6) == 0) {
+				fprintf(stderr, "Server got bad message length. ");
+				goto codec_revert;
+			} else if (strncmp("BADIP", in, 5) == 0) {
+				fprintf(stderr, "Server rejected sender IP address. ");
+				goto codec_revert;
+			} else if (strncmp("BADCODEC", in, 8) == 0) {
+				fprintf(stderr, "Server rejected lazy mode. ");
+				goto codec_revert;
+			} else if (strncmp("Lazy", in, 4) == 0) {
+				fprintf(stderr, "Server switched to lazy mode\n");
+				lazymode = 1;
+				return;
 			}
-		/*XXX END adjust indent 1 tab back*/
+		}
 
 		fprintf(stderr, "Retrying lazy mode switch...\n");
 	}
@@ -2242,14 +2230,12 @@ handshake_lazyoff(int dns_fd)
 
 		read = handshake_waitdns(dns_fd, in, sizeof(in), 'o', 'O', 1);
 
-		/*XXX START adjust indent 2 tabs back*/
-				if (read == 9 && strncmp("Immediate", in, 9) == 0) {
-					warnx("Server switched back to legacy mode.\n");
-					lazymode = 0;
-					selecttimeout = 1;
-					return;
-				}
-		/*XXX END adjust indent 2 tabs back*/
+		if (read == 9 && strncmp("Immediate", in, 9) == 0) {
+			warnx("Server switched back to legacy mode.\n");
+			lazymode = 0;
+			selecttimeout = 1;
+			return;
+		}
 	}
 	if (!running)
 		return;
@@ -2304,28 +2290,26 @@ fragsize_check(char *in, int read, int proposed_fragsize, int *max_fragsize)
 	okay = 1;
 	v = in[3] & 0xff;
 
-	/*XXX START adjust indent 1 tab back*/
-		for (i = 3; i < read; i++, v = (v + 107) & 0xff)
-			if ((in[i] & 0xff) != v) {
-				okay = 0;
-				break;
-			}
-
-		if (okay) {
-			fprintf(stderr, "%d ok.. ", acked_fragsize);
-			fflush(stderr);
-			*max_fragsize = acked_fragsize;
-			return 1;
-		} else {
-			if (downenc != ' ' && downenc != 'T') {
-				fprintf(stderr, "%d corrupted at %d.. (Try -O Base32)\n", acked_fragsize, i);
-			} else {
-				fprintf(stderr, "%d corrupted at %d.. ", acked_fragsize, i);
-			}
-			fflush(stderr);
-			return 1;
+	for (i = 3; i < read; i++, v = (v + 107) & 0xff)
+		if ((in[i] & 0xff) != v) {
+			okay = 0;
+			break;
 		}
-	/*XXX END adjust indent 1 tab back*/
+
+	if (okay) {
+		fprintf(stderr, "%d ok.. ", acked_fragsize);
+		fflush(stderr);
+		*max_fragsize = acked_fragsize;
+		return 1;
+	} else {
+		if (downenc != ' ' && downenc != 'T') {
+			fprintf(stderr, "%d corrupted at %d.. (Try -O Base32)\n", acked_fragsize, i);
+		} else {
+			fprintf(stderr, "%d corrupted at %d.. ", acked_fragsize, i);
+		}
+		fflush(stderr);
+		return 1;
+	}
 
 	/* notreached */
 	return 1;
@@ -2352,13 +2336,11 @@ handshake_autoprobe_fragsize(int dns_fd)
 
 			read = handshake_waitdns(dns_fd, in, sizeof(in), 'r', 'R', 1);
 				
-			/*XXX START adjust indent 1 tab back*/
-				if (read > 0) {
-					/* We got a reply */
-					if (fragsize_check(in, read, proposed_fragsize, &max_fragsize) == 1)
-						break;
-				}
-			/*XXX END adjust indent 1 tab back*/
+			if (read > 0) {
+				/* We got a reply */
+				if (fragsize_check(in, read, proposed_fragsize, &max_fragsize) == 1)
+					break;
+			}
 
 			fprintf(stderr, ".");
 			fflush(stderr);
@@ -2422,22 +2404,20 @@ handshake_set_fragsize(int dns_fd, int fragsize)
 
 		read = handshake_waitdns(dns_fd, in, sizeof(in), 'n', 'N', i+1);
 
-		/*XXX START adjust indent 1 tab back*/			
-			if (read > 0) {
-				int accepted_fragsize;
+		if (read > 0) {
+			int accepted_fragsize;
 
-				if (strncmp("BADFRAG", in, 7) == 0) {
-					fprintf(stderr, "Server rejected fragsize. Keeping default.");
-					return;
-				} else if (strncmp("BADIP", in, 5) == 0) {
-					fprintf(stderr, "Server rejected sender IP address.\n");
-					return;
-				}
-
-				accepted_fragsize = ((in[0] & 0xff) << 8) | (in[1] & 0xff);
+			if (strncmp("BADFRAG", in, 7) == 0) {
+				fprintf(stderr, "Server rejected fragsize. Keeping default.");
+				return;
+			} else if (strncmp("BADIP", in, 5) == 0) {
+				fprintf(stderr, "Server rejected sender IP address.\n");
 				return;
 			}
-		/*XXX END adjust indent 1 tab back*/
+
+			accepted_fragsize = ((in[0] & 0xff) << 8) | (in[1] & 0xff);
+			return;
+		}
 
 		fprintf(stderr, "Retrying set fragsize...\n");
 	}
diff --git a/src/dns.c b/src/dns.c
index eb29403..a81ae09 100644
--- a/src/dns.c
+++ b/src/dns.c
@@ -207,15 +207,13 @@ dns_encode(char *buf, size_t buflen, struct query *q, qr_t qr, char *data, size_
 		   (even CNAME/A/MX, 255+255+header would be >512) */
 		if (dnsc_use_edns0) {
 			header->arcount = htons(1);
-			/*XXX START adjust indent 1 tab forward*/
-		CHECKLEN(11);
-		putbyte(&p, 0x00);    /* Root */
-		putshort(&p, 0x0029); /* OPT */
-		putshort(&p, 0x1000); /* Payload size: 4096 */
-		putshort(&p, 0x0000); /* Higher bits/edns version */
-		putshort(&p, 0x8000); /* Z */
-		putshort(&p, 0x0000); /* Data length */
-			/*XXX END adjust indent 1 tab forward*/
+			CHECKLEN(11);
+			putbyte(&p, 0x00);    /* Root */
+			putshort(&p, 0x0029); /* OPT */
+			putshort(&p, 0x1000); /* Payload size: 4096 */
+			putshort(&p, 0x0000); /* Higher bits/edns version */
+			putshort(&p, 0x8000); /* Z */
+			putshort(&p, 0x0000); /* Data length */
 		}
 
 		break;
diff --git a/src/iodined.c b/src/iodined.c
index 3a15ba1..3bfbd4e 100644
--- a/src/iodined.c
+++ b/src/iodined.c
@@ -1742,31 +1742,29 @@ handle_full_packet(int tun_fd, int dns_fd, int userid)
 			write_tun(tun_fd, out, outlen);
 		} else {
 			/* send the compressed(!) packet to other client */
-		/*XXX START adjust indent 1 tab forward*/
-		if (users[touser].conn == CONN_DNS_NULL) {
-			if (users[touser].outpacket.len == 0) {
-				start_new_outpacket(touser,
-					users[userid].inpacket.data,
-					users[userid].inpacket.len);
+			if (users[touser].conn == CONN_DNS_NULL) {
+				if (users[touser].outpacket.len == 0) {
+					start_new_outpacket(touser,
+						users[userid].inpacket.data,
+						users[userid].inpacket.len);
 
-				/* Start sending immediately if query is waiting */
-				if (users[touser].q_sendrealsoon.id != 0)
-					send_chunk_or_dataless(dns_fd, touser, &users[touser].q_sendrealsoon);
-				else if (users[touser].q.id != 0)
-					send_chunk_or_dataless(dns_fd, touser, &users[touser].q);
+					/* Start sending immediately if query is waiting */
+					if (users[touser].q_sendrealsoon.id != 0)
+						send_chunk_or_dataless(dns_fd, touser, &users[touser].q_sendrealsoon);
+					else if (users[touser].q.id != 0)
+						send_chunk_or_dataless(dns_fd, touser, &users[touser].q);
 #ifdef OUTPACKETQ_LEN
-			} else {
-				save_to_outpacketq(touser,
-					users[userid].inpacket.data,
-					users[userid].inpacket.len);
+				} else {
+					save_to_outpacketq(touser,
+						users[userid].inpacket.data,
+						users[userid].inpacket.len);
 #endif
+				}
+			} else{ /* CONN_RAW_UDP */
+				send_raw(dns_fd, users[userid].inpacket.data,
+					 users[userid].inpacket.len, touser,
+					 RAW_HDR_CMD_DATA, &users[touser].q);
 			}
-		} else{ /* CONN_RAW_UDP */
-			send_raw(dns_fd, users[userid].inpacket.data,
-				 users[userid].inpacket.len, touser,
-				 RAW_HDR_CMD_DATA, &users[touser].q);
-		}
-		/*XXX END adjust indent 1 tab forward*/
 		}
 	} else {
 		if (debug >= 1)