Function returns pointer to trash, and free locks the terminal

Question

Could someone explain to me the reason when I give free in the pontaux Windows terminal stops responding? And why when I allocate pontmaior my result comes as memory junk?

The function returns a pointer to the highest value of a vector.

Code:

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

int* retornamaior (int *vetor, int tamanho) {

    int i, maior = vetor[0];

    for (i=0; i<tamanho; i++) {
        if (vetor[i]>=maior)
            maior = vetor[i];   
        }

    int *pontmaior = &maior;

    return pontmaior;
}

int main (int argc, char *argv[]) {

    int *vetor, tamanho, i;

    printf ("Digite o tamanho da sequência: ");
    scanf ("%d", &tamanho);

    vetor = (int*)malloc(tamanho*sizeof(int));

    printf ("Digite a sequência: ");

    for (i=0; i<tamanho; i++)
        scanf ("%d", &vetor[i]);

    int *pontaux = retornamaior (vetor,tamanho);

    printf ("Maior: %d\n", *pontaux);

    free (vetor);

    return 0;   
}

Answer 1

The error seems to be here:

int *pontmaior = &maior;

You are returning a pointer to the "larger" stack variable, which no longer exists when used by the main().

You should return a simple integer pointing out which item of the "vector" matrix is the largest. It is safer, avoid this lot of pointer manipulations.

Would something like this:

int retornamaior (int *vetor, int tamanho)
{
        int i, maior = 0;
        for (i=0; i<tamanho; i++) {
                if (vetor[i]>=vetor[maior])
                        maior = i;
        }
        return maior;
}

int main (int argc, char *argv[])
{
        int *vetor, tamanho, i;

        printf ("Digite o tamanho da sequência: ");
        scanf ("%d", &tamanho);

        vetor = (int*)malloc(tamanho*sizeof(int));

        printf ("Digite a sequência: ");

        for (i=0; i<tamanho; i++)
                scanf ("%d", &vetor[i]);

        int pontaux = retornamaior (vetor,tamanho);

        printf ("Maior: %d\n", vetor[pontaux]);

        free (vetor);

        return 0;
}

If you make a point of returning a pointer:

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

int* retornamaior (int *vetor, int tamanho) {

    int i, *maior = &vetor[0];

    for (i=0; i<tamanho; i++) {
        if (vetor[i]>=*maior)
            maior = &vetor[i];
        }

    return maior;
}

int main (int argc, char *argv[]) {

    int *vetor, tamanho, i;

    printf ("Digite o tamanho da sequência: ");
    scanf ("%d", &tamanho);

    vetor = (int*)malloc(tamanho*sizeof(int));

    printf ("Digite a sequência: ");

    for (i=0; i<tamanho; i++)
        scanf ("%d", &vetor[i]);

    int *pontaux = retornamaior (vetor,tamanho);

    printf ("Maior: %d\n", *pontaux);

    free (vetor);

    return 0;
}

Answer 2

I saw the epx response and I realized that it got a little lost to understand why the following piece of code

int *pontmaior = &maior;

be wrong, what happens is that when you declare int *pointmaior within your function you are declaring a variable that will only exist within the scope of the function int* retornamaior (int *vetor, int tamanho), that is, the memory area intended for this variable will be released by the system at the end of the execution of this function, thus, when executing the line

printf ("Maior: %d\n", *pontaux);

you will already attempt to access an invalid memory area.