diff options
| author | Corey Farrell <git@cfware.com> | 2017-11-18 21:02:17 -0500 |
|---|---|---|
| committer | Corey Farrell <git@cfware.com> | 2017-11-18 20:05:03 -0600 |
| commit | 85d675b14c7f946b86cc371a43aaa7f5d314d8c2 (patch) | |
| tree | 85e067e1099258c9742670f3ab4f1bc0b84a1eea /res/res_fax.c | |
| parent | 8fdc7fd1d3e77ae171671231e318d15710a0f715 (diff) | |
res_fax: Remove checks for unsigned values being >= 0.
It's impossible for gwtimeout or fdtimeout to be less than 0 because
they are unsigned int's. Remove checks and unreachable branches.
Change-Id: Ib2286960621e6ee245e40013c84986143302bc78
Diffstat (limited to 'res/res_fax.c')
| -rw-r--r-- | res/res_fax.c | 15 |
1 files changed, 2 insertions, 13 deletions
diff --git a/res/res_fax.c b/res/res_fax.c index b25b6d16d..4a1c0846f 100644 --- a/res/res_fax.c +++ b/res/res_fax.c @@ -4561,12 +4561,7 @@ static int acf_faxopt_write(struct ast_channel *chan, const char *cmd, char *dat unsigned int gwtimeout; if (sscanf(timeout, "%30u", &gwtimeout) == 1) { - if (gwtimeout >= 0) { - details->gateway_timeout = gwtimeout * 1000; - } else { - ast_log(LOG_WARNING, "%s(%s) timeout cannot be negative. Ignoring timeout\n", - cmd, data); - } + details->gateway_timeout = gwtimeout * 1000; } else { ast_log(LOG_WARNING, "Unsupported timeout '%s' passed to FAXOPT(%s).\n", timeout, data); } @@ -4605,13 +4600,7 @@ static int acf_faxopt_write(struct ast_channel *chan, const char *cmd, char *dat if (details->faxdetect_id < 0) { if (timeout) { if (sscanf(timeout, "%30u", &fdtimeout) == 1) { - if (fdtimeout >= 0) { - fdtimeout *= 1000; - } else { - ast_log(LOG_WARNING, "%s(%s) timeout cannot be negative. Ignoring timeout\n", - cmd, data); - fdtimeout = 0; - } + fdtimeout *= 1000; } else { ast_log(LOG_WARNING, "Unsupported timeout '%s' passed to FAXOPT(%s).\n", timeout, data); |