diff options
author | Philipp Maier <pmaier@sysmocom.de> | 2023-03-06 14:26:31 +0100 |
---|---|---|
committer | laforge <laforge@osmocom.org> | 2023-03-07 12:54:58 +0000 |
commit | 280ad4d966d5863f96eaf843db5549431604a9a9 (patch) | |
tree | a562a663a9c19ef128bcd514b3abab3be6b56b72 | |
parent | 7630b88fb591f22eafb80f4b1e717a4622b6f783 (diff) |
pcu_sock: rename rc to fd
The variable rc holds the return code of accept(), which returns a file
descriptor on success. So lets call the variable "fd" to make this
clear.
Change-Id: Ic8d22c2af18477f110a3a9115434314ebca95b25
-rw-r--r-- | src/osmo-bsc/pcu_sock.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/src/osmo-bsc/pcu_sock.c b/src/osmo-bsc/pcu_sock.c index 486f364bf..1d7b0617e 100644 --- a/src/osmo-bsc/pcu_sock.c +++ b/src/osmo-bsc/pcu_sock.c @@ -911,14 +911,14 @@ static int pcu_sock_accept(struct osmo_fd *bfd, unsigned int flags) struct gsm_bts_trx *trx; int j; socklen_t len; - int rc; + int fd; /* FIXME: allow multiple BTS */ bts = llist_entry(state->net->bts_list.next, struct gsm_bts, list); len = sizeof(un_addr); - rc = accept(bfd->fd, (struct sockaddr *) &un_addr, &len); - if (rc < 0) { + fd = accept(bfd->fd, (struct sockaddr *) &un_addr, &len); + if (fd < 0) { LOG_BTS(bts, DPCU, LOGL_ERROR, "Failed to accept a new connection\n"); return -1; } @@ -927,11 +927,11 @@ static int pcu_sock_accept(struct osmo_fd *bfd, unsigned int flags) LOG_BTS(bts, DPCU, LOGL_NOTICE, "PCU connects but we already have another active connection ?!?\n"); /* We already have one PCU connected, this is all we support */ osmo_fd_read_disable(&state->listen_bfd); - close(rc); + close(fd); return 0; } - osmo_fd_setup(conn_bfd, rc, OSMO_FD_READ, pcu_sock_cb, state, 0); + osmo_fd_setup(conn_bfd, fd, OSMO_FD_READ, pcu_sock_cb, state, 0); if (osmo_fd_register(conn_bfd) != 0) { LOG_BTS(bts, DPCU, LOGL_ERROR, "Failed to register new connection fd\n"); |