pbx.
*/
/* first, get the feature string, if it exists */
- struct ast_call_feature *feat = ast_find_call_feature("automon");
+ struct ast_call_feature *feat;
int j;
struct ast_frame f = { AST_FRAME_DTMF, };
-
+
+ ast_rdlock_call_features();
+ feat = ast_find_call_feature("automon");
if (!feat || ast_strlen_zero(feat->exten)) {
ast_log(LOG_WARNING,"Recording requested, but no One Touch Monitor registered. (See features.conf)\n");
/* 403 means that we don't support this feature, so don't request it again */
transmit_response(p, "403 Forbidden", req);
+ ast_unlock_call_features();
return;
}
/* OEJ: Why is the DTMF code included in the record section? */
if (sipdebug)
ast_verbose("* DTMF-relay event received: %c\n", f.subclass);
}
+ ast_unlock_call_features();
#ifdef DISABLED_CODE
/* And feat isn't used here - Is this code tested at all???
We just send a reply ...
/*! \brief look for a call feature entry by its sname
\param name a string ptr, should match "automon", "blindxfer", "atxfer", etc. */
-struct ast_call_feature *ast_find_call_feature(char *name);
+struct ast_call_feature *ast_find_call_feature(const char *name);
+
+void ast_rdlock_call_features(void);
+void ast_unlock_call_features(void);
#endif /* _AST_FEATURES_H */
return tmp;
}
+void ast_rdlock_call_features(void)
+{
+ ast_rwlock_rdlock(&features_lock);
+}
+
+void ast_unlock_call_features(void)
+{
+ ast_rwlock_unlock(&features_lock);
+}
+
/*! \brief find a call feature by name */
-struct ast_call_feature *ast_find_call_feature(char *name)
+struct ast_call_feature *ast_find_call_feature(const char *name)
{
int x;
for (x = 0; x < FEATURES_COUNT; x++) {
- if (!strcasecmp(name, builtin_features[x].sname)) {
+ if (!strcasecmp(name, builtin_features[x].sname))
return &builtin_features[x];
- }
}
return NULL;
}