Я думал, что есть проблема вторжения памяти, правильно? - PullRequest
Новая
       8

Я думал, что есть проблема вторжения памяти, правильно?

0 голосов
/ 02 февраля 2020

Как показано ниже, мой код: 

int     *text(char *str)
{
    int     *cond;
    int     *temp;
    int     cond_size;
    int     num;
    int     i;

    cond_size = -1;
    cond = malloc(sizeof(int) * 1);
    *cond = 0;
    while (*str != '\0')
    {
        if (*str == ' ')
            str++;
        num = 0;
        while (*str >= '0' && *str <= '9')
            num = num * 10 + *(str++) - '0';
        temp = cond;
        cond = malloc(sizeof(int) * (++cond_size));
        i = -1;
        while (++i < cond_size)
            cond[i] = temp[i];
        cond[i] = num;
        free(temp);
    }
    g_size = (i + 1) / 4;
    return (cond);
}

И моя основная функция: 

int     *text(char *str);

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

int     g_size = 0;

int     main(void)
{
    int     *test;
    int     i;

    i = 0;
    test = text(" 4 3 2 1 1 2 2 2 4 3 2 1 1 2 2 2");
    while(i < g_size)
    {
        printf("\n%d\n", test[i]);
        i++;
    }
}

Со строкой ввода 4 3 2 1 1 2 2 2 4 3 2 1 ... выводится следующий вывод: 

==============
|  4 3 2 1   |
|4        '1'| <==
|3         2 |  
|2         2 |
|1         2 |
|  1 2 2 2   |
==============

Однако, как я проверял (<==), </p>

'1' был напечатан как '4' ,

, и это не так правильно, как я и ожидал.

Есть ли в моем коде вторжение памяти при использовании mallo c или другая ошибка?

1 Ответ

0 голосов
/ 02 февраля 2020

Я вижу некоторые проблемы:

  • cond_size равно -1: 

    cond_size = -1; // Here is problem
cond = malloc(sizeof(int) * 1);
*cond = 0;
while (*str != '\0')
{
    if (*str == ' ')
        str++;
    num = 0;
    while (*str >= '0' && *str <= '9')
       num = num * 10 + *(str++) - '0';
    temp = cond;
    cond = malloc(sizeof(int) * (++cond_size)); // you are trying allocate memory with size 0
    i = -1;
    while (++i < cond_size)
        cond[i] = temp[i];
    cond[i] = num; // you are writing to not allocated memory

    Вы можете найти больше о mallo c .

  • температура вне диапазона: 

    temp = cond;
cond = malloc(sizeof(int) * (++cond_size));
i = -1;
while (++i < cond_size)
    cond[i] = temp[i]; // temp size is cond_size-1

    Таким образом, вы должны ограничить l oop значением cond_size-1.

Пожалуйста, обратите внимание на исправленную версию. 

<script src="//onlinegdb.com/embed/js/SJADLQEM8?theme=dark"></script>
...