You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by zw...@apache.org on 2016/10/27 08:32:10 UTC
[trafficserver] branch master updated: TS-5012 CID 1267812
Logically Dead Code fixed
This is an automated email from the ASF dual-hosted git repository.
zwoop pushed a commit to branch master
in repository https://git-dual.apache.org/repos/asf/trafficserver.git
The following commit(s) were added to refs/heads/master by this push:
new 3f8f680 TS-5012 CID 1267812 Logically Dead Code fixed
3f8f680 is described below
commit 3f8f680dac047aa66469d711f3c85d075a2471c1
Author: Nathan Garabedian <ng...@users.noreply.github.com>
AuthorDate: Wed Oct 26 23:54:32 2016 +0000
TS-5012 CID 1267812 Logically Dead Code fixed
---
mgmt/ProxyConfig.cc | 5 +----
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/mgmt/ProxyConfig.cc b/mgmt/ProxyConfig.cc
index f26e02b..b59b445 100644
--- a/mgmt/ProxyConfig.cc
+++ b/mgmt/ProxyConfig.cc
@@ -190,12 +190,9 @@ ConfigProcessor::release(unsigned int id, ConfigInfo *info)
{
int idx;
- ink_assert(id != 0);
- ink_assert(id <= MAX_CONFIGS);
-
if (id == 0 || id > MAX_CONFIGS) {
// nothing to delete since we have an invalid index
- return;
+ ink_abort("released an invalid id '%u'", id);
}
idx = id - 1;
--
To stop receiving notification emails like this one, please contact
['"commits@trafficserver.apache.org" <co...@trafficserver.apache.org>'].