diff options
| author | file <file@f38db490-d61c-443f-a65b-d21fe96a405b> | 2007-06-04 17:41:59 +0000 |
|---|---|---|
| committer | file <file@f38db490-d61c-443f-a65b-d21fe96a405b> | 2007-06-04 17:41:59 +0000 |
| commit | 977074e966942456e28c73ec2d6a6468ab407e02 (patch) | |
| tree | 77e43d3ed9ece124bb946144f196c63640985ef0 /res | |
| parent | fca9c78f0da73586e577347e73559109ae9e158a (diff) | |
Returning a value that indicates the parking of a call was a success when it really wasn't (because the parking slot selected was in use) is the wrong thing to do. (issue #9723 reported by mdu113)
git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@67064 f38db490-d61c-443f-a65b-d21fe96a405b
Diffstat (limited to 'res')
| -rw-r--r-- | res/res_features.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/res/res_features.c b/res/res_features.c index 879b2b591..9ffc2861d 100644 --- a/res/res_features.c +++ b/res/res_features.c @@ -328,7 +328,7 @@ int ast_park_call(struct ast_channel *chan, struct ast_channel *peer, int timeou ast_mutex_unlock(&parking_lock); free(pu); ast_log(LOG_WARNING, "Requested parking extension already exists: %s@%s\n", parkingexten, parking_con); - return 0; /* Continue execution if possible */ + return -1; /* We failed to park this call, plain and simple so we need to error out */ } ast_copy_string(pu->parkingexten, parkingexten, sizeof(pu->parkingexten)); x = atoi(parkingexten); |