mgcp_trunk: use unsigned int instead of int as trunk_nr
the trunk_nr is in struct mgcp_trunk. The trunk number can not be
negative and there is no magic value that makes use of the fact that it
could be negative. Lets use unsigned int to make this less irretating.
Change-Id: I5d0e1d76adb8c92d84331a0aca2496908e41d621
Related: SYS#5535
diff --git a/src/libosmo-mgcp/mgcp_trunk.c b/src/libosmo-mgcp/mgcp_trunk.c
index 08f99b3..27663b4 100644
--- a/src/libosmo-mgcp/mgcp_trunk.c
+++ b/src/libosmo-mgcp/mgcp_trunk.c
@@ -40,7 +40,7 @@
* \param[in] ttype trunk type.
* \param[in] nr trunk number.
* \returns pointer to allocated trunk, NULL on failure. */
-struct mgcp_trunk *mgcp_trunk_alloc(struct mgcp_config *cfg, enum mgcp_trunk_type ttype, int nr)
+struct mgcp_trunk *mgcp_trunk_alloc(struct mgcp_config *cfg, enum mgcp_trunk_type ttype, unsigned int nr)
{
struct mgcp_trunk *trunk;
@@ -171,7 +171,7 @@
* \param[in] ttype trunk type.
* \param[in] nr trunk number.
* \returns pointer to trunk configuration, NULL on error. */
-struct mgcp_trunk *mgcp_trunk_by_num(const struct mgcp_config *cfg, enum mgcp_trunk_type ttype, int nr)
+struct mgcp_trunk *mgcp_trunk_by_num(const struct mgcp_config *cfg, enum mgcp_trunk_type ttype, unsigned int nr)
{
struct mgcp_trunk *trunk;
@@ -184,9 +184,9 @@
}
/* Made public for unit-testing, do not use from outside this file */
-int e1_trunk_nr_from_epname(const char *epname)
+int e1_trunk_nr_from_epname(unsigned int *trunk_nr, const char *epname)
{
- unsigned long int trunk_nr;
+ unsigned long trunk_nr_temp;
size_t prefix_len;
char *str_trunk_nr_end;
@@ -195,13 +195,15 @@
return -EINVAL;
errno = 0;
- trunk_nr = strtoul(epname + prefix_len, &str_trunk_nr_end, 10);
- if (errno == ERANGE || trunk_nr > 64
+ trunk_nr_temp = strtoul(epname + prefix_len, &str_trunk_nr_end, 10);
+ if (errno == ERANGE || trunk_nr_temp > 64
|| epname + prefix_len == str_trunk_nr_end
|| str_trunk_nr_end[0] != '/')
return -EINVAL;
- else
- return trunk_nr;
+ else {
+ *trunk_nr = (unsigned int)trunk_nr_temp;
+ return 0;
+ }
}
/*! Find a trunk by the trunk prefix in the endpoint name.
@@ -212,7 +214,8 @@
{
size_t prefix_len;
char epname_lc[MGCP_ENDPOINT_MAXLEN];
- int trunk_nr;
+ unsigned int trunk_nr;
+ int rc;
osmo_str_tolower_buf(epname_lc, sizeof(epname_lc), epname);
epname = epname_lc;
@@ -222,8 +225,8 @@
return mgcp_trunk_by_num(cfg, MGCP_TRUNK_VIRTUAL, MGCP_VIRT_TRUNK_ID);
}
- trunk_nr = e1_trunk_nr_from_epname(epname);
- if (trunk_nr >= 0)
+ rc = e1_trunk_nr_from_epname(&trunk_nr, epname);
+ if (rc == 0)
return mgcp_trunk_by_num(cfg, MGCP_TRUNK_E1, trunk_nr);
/* Earlier versions of osmo-mgw were accepting endpoint names