]> git.ipfire.org Git - thirdparty/asterisk.git/commitdiff
Fix incorrect sizeof() usage in features.c.
authorRussell Bryant <russell@russellbryant.com>
Wed, 14 Mar 2012 00:21:18 +0000 (00:21 +0000)
committerRussell Bryant <russell@russellbryant.com>
Wed, 14 Mar 2012 00:21:18 +0000 (00:21 +0000)
This didn't actually result in a bug anywhere, luckily.  The only place
where the result of these memcpys was used is in app_dial, and the only
field that it read out of ast_call_feature was the first one, which is an
int, so these memcpys always copied just enough to avoid a problem.
........

Merged revisions 359069 from http://svn.asterisk.org/svn/asterisk/branches/1.8

git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/10@359072 65c4cc65-6c06-0410-ace0-fbb531ad65f3

main/features.c

index f38ceadba4cac6b88acc0f3feeb623cb70a06e11..dcf09e1bd6e47f4d3da7ca6981dc012906b530b5 100644 (file)
@@ -3163,7 +3163,7 @@ static int feature_interpret_helper(struct ast_channel *chan, struct ast_channel
                                        res = builtin_features[x].operation(chan, peer, config, code, sense, NULL);
                                }
                                if (feature) {
-                                       memcpy(feature, &builtin_features[x], sizeof(feature));
+                                       memcpy(feature, &builtin_features[x], sizeof(*feature));
                                }
                                feature_detected = 1;
                                break;
@@ -3193,7 +3193,7 @@ static int feature_interpret_helper(struct ast_channel *chan, struct ast_channel
                                        if (operation) {
                                                res = fge->feature->operation(chan, peer, config, code, sense, fge->feature);
                                        }
-                                       memcpy(feature, fge->feature, sizeof(feature));
+                                       memcpy(feature, fge->feature, sizeof(*feature));
                                        if (res != AST_FEATURE_RETURN_KEEPTRYING) {
                                                AST_RWLIST_UNLOCK(&feature_groups);
                                                break;
@@ -3226,7 +3226,7 @@ static int feature_interpret_helper(struct ast_channel *chan, struct ast_channel
                                res = tmpfeature->operation(chan, peer, config, code, sense, tmpfeature);
                        }
                        if (feature) {
-                               memcpy(feature, tmpfeature, sizeof(feature));
+                               memcpy(feature, tmpfeature, sizeof(*feature));
                        }
                        if (res != AST_FEATURE_RETURN_KEEPTRYING) {
                                AST_RWLIST_UNLOCK(&feature_list);