[PATCH] osmo-bsc[master]: bsc_nat: forward_to_bsc: remove one level of indentation

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

Pau Espin Pedrol gerrit-no-reply at lists.osmocom.org
Wed Apr 11 14:41:22 UTC 2018


Review at  https://gerrit.osmocom.org/7756

bsc_nat: forward_to_bsc: remove one level of indentation

Change-Id: I12d86a45a6b2cdc3af7a7b6c28f8f337fa95c903
---
M src/osmo-bsc_nat/bsc_nat_ctrl.c
1 file changed, 50 insertions(+), 49 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/osmo-bsc refs/changes/56/7756/1

diff --git a/src/osmo-bsc_nat/bsc_nat_ctrl.c b/src/osmo-bsc_nat/bsc_nat_ctrl.c
index 9450741..5f82a41 100644
--- a/src/osmo-bsc_nat/bsc_nat_ctrl.c
+++ b/src/osmo-bsc_nat/bsc_nat_ctrl.c
@@ -236,57 +236,58 @@
 			continue;
 		if (!bsc->authenticated)
 			continue;
-		if (bsc->cfg->nr == nr) {
-			/* Add pending command to list */
-			pending = talloc_zero(bsc, struct bsc_cmd_list);
-			if (!pending) {
-				cmd->reply = "OOM";
-				goto err;
-			}
+		if (bsc->cfg->nr != nr)
+			continue;
 
-			pending->nat_id = get_next_free_bsc_id(bsc);
-			if (pending->nat_id < 0) {
-				cmd->reply = "No free ID found";
-				goto err;
-			}
-
-			bsc_cmd = ctrl_cmd_cpy(bsc, cmd);
-			if (!bsc_cmd) {
-				cmd->reply = "Could not forward command";
-				goto err;
-			}
-
-			talloc_free(bsc_cmd->id);
-			bsc_cmd->id = talloc_asprintf(bsc_cmd, "%i", pending->nat_id);
-			if (!bsc_cmd->id) {
-				cmd->reply = "OOM";
-				goto err;
-			}
-
-			talloc_free(bsc_cmd->variable);
-			bsc_cmd->variable = talloc_strdup(bsc_cmd, bsc_variable);
-			if (!bsc_cmd->variable) {
-				cmd->reply = "OOM";
-				goto err;
-			}
-
-			if (ctrl_cmd_send(&bsc->write_queue, bsc_cmd)) {
-				cmd->reply = "Sending failed";
-				goto err;
-			}
-			pending->ccon = cmd->ccon;
-			pending->ccon->closed_cb = ctrl_conn_closed_cb;
-			pending->cmd = cmd;
-
-			/* Setup the timeout */
-			osmo_timer_setup(&pending->timeout, pending_timeout_cb,
-					 pending);
-			/* TODO: Make timeout configurable */
-			osmo_timer_schedule(&pending->timeout, 10, 0);
-			llist_add_tail(&pending->list_entry, &bsc->cmd_pending);
-
-			goto done;
+		/* Add pending command to list */
+		pending = talloc_zero(bsc, struct bsc_cmd_list);
+		if (!pending) {
+			cmd->reply = "OOM";
+			goto err;
 		}
+
+		pending->nat_id = get_next_free_bsc_id(bsc);
+		if (pending->nat_id < 0) {
+			cmd->reply = "No free ID found";
+			goto err;
+		}
+
+		bsc_cmd = ctrl_cmd_cpy(bsc, cmd);
+		if (!bsc_cmd) {
+			cmd->reply = "Could not forward command";
+			goto err;
+		}
+
+		talloc_free(bsc_cmd->id);
+		bsc_cmd->id = talloc_asprintf(bsc_cmd, "%i", pending->nat_id);
+		if (!bsc_cmd->id) {
+			cmd->reply = "OOM";
+			goto err;
+		}
+
+		talloc_free(bsc_cmd->variable);
+		bsc_cmd->variable = talloc_strdup(bsc_cmd, bsc_variable);
+		if (!bsc_cmd->variable) {
+			cmd->reply = "OOM";
+			goto err;
+		}
+
+		if (ctrl_cmd_send(&bsc->write_queue, bsc_cmd)) {
+			cmd->reply = "Sending failed";
+			goto err;
+		}
+		pending->ccon = cmd->ccon;
+		pending->ccon->closed_cb = ctrl_conn_closed_cb;
+		pending->cmd = cmd;
+
+		/* Setup the timeout */
+		osmo_timer_setup(&pending->timeout, pending_timeout_cb,
+				 pending);
+		/* TODO: Make timeout configurable */
+		osmo_timer_schedule(&pending->timeout, 10, 0);
+		llist_add_tail(&pending->list_entry, &bsc->cmd_pending);
+
+		goto done;
 	}
 	/* We end up here if there's no bsc to handle our LAC */
 	cmd->reply = "no BSC with this nr";

-- 
To view, visit https://gerrit.osmocom.org/7756
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I12d86a45a6b2cdc3af7a7b6c28f8f337fa95c903
Gerrit-PatchSet: 1
Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Owner: Pau Espin Pedrol <pespin at sysmocom.de>



More information about the gerrit-log mailing list