C++ 为什么使用 C 引用这个 char 数组会导致 Stack smasing

该函数必须构建包含6个字符的字符串"10 20"，该6个字符包括终止空字符'\0'.

但是您正在try 将该字符串存储在一个只有4个字符的数组中

char str[] = "1 2";

由于这些声明

strcat(arr,one);                // writing the multiplied values to the string
strcat(arr, " ");
strcat(arr,two);

因此，该函数已经调用了未定义的行为.

另一个问题是在这个memset人的电话中:

memset(arr,0,sizeof(arr));

函数内的变量arr具有指针类型char *.如果sizeof( char * )等于8，则再次try 写入数组外部的存储器.

并且该函数不应依赖于此声明中使用的幻数(如2

int num_arr[2];

您应该始终try 编写更通用的函数.

要解决这个问题，您应该在函数内动态分配一个新的字符数组来存储结果字符串，并从函数返回指向该数组的指针.

另外，请注意，这样写起来会更清楚和正确

read_and_mul( str, 10 );

而不是

read_and_mul((char *) &str, 10);

下面是一个演示程序，它展示了一种解决该任务的可能方法.

#include <stdio.h>
#include <stdlib.h>

char * read_and_mul( const char *s, int scale )
{
    size_t n = 0;
    size_t length = 0;

    const char *tmp = s;
    int value;

    for (char *endptr; value = strtol( tmp, &endptr, 10 ), endptr != tmp; tmp = endptr)
    {
        ++n;
        length += snprintf( NULL, 0, "%d", value * scale );
    }

    length += n == 0 ? 1 : n;

    char *result = calloc( length, sizeof( char ) );

    if (result != NULL)
    {
        const char *tmp = s;
        int first = 1;

        for (char *pos = result, *endptr; value = strtol( tmp, &endptr, 10 ), endptr != tmp; tmp = endptr)
        {
            if (!first)
            {
                *pos++ = ' ';
            }
            else
            {
                first = 0;
            }

            pos += sprintf( pos, "%d", value * scale );
        }
    }

    return result;
}

int main( void )
{
    char s[] = "1 2 3 4 5 6 7 8 9 10";

    char *result = read_and_mul( s, 10 );

    if (result) printf( "\"%s\"\n", result);

    free( result );
}

程序输出为

"10 20 30 40 50 60 70 80 90 100"

因为通常情况下，两个整数的乘法可能会导致溢出，为了避免这种情况，您可以更改以下语句

length += snprintf( NULL, 0, "%d", value * scale );
pos += sprintf( pos, "%d", value * scale );

至以下各项

length += snprintf( NULL, 0, "%lld", ( long long int )value * scale );
pos += sprintf( pos, "%lld", ( long long int )value * scale );