Почему в программе утечка памяти?

Question

[Hello World]

Я написал программу, которая удаляет пробелы:

код

#include <stdint.h>
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
char *inputString(size_t size);
char *CutSpaces(char *str);

int main(){
	char *str, *new_str;
	
	str = inputString(10);	
	new_str = CutSpaces(str);	
	
	printf("%s\n", new_str);


	free(new_str);	
	free(str);
	return 0;
}

char *inputString(size_t size){
	char *str;
	int ch;
	size_t len = 0;
	str = calloc(size, sizeof(char));
	if (!str) return str;
	while ((ch = getchar()) != EOF && ch != '\n' ){
		str[len++]=ch;
		if (len==size){
			char *tmp  = realloc(str, sizeof(char)*(size+=16));
			if (!tmp) 
				return str;
			else 
				str = tmp;
		}
	
		
	}
	str[len++] = '\0';
	return realloc(str, sizeof(char)*len);
}



char *CutSpaces(char *str){
	char *dst = calloc(strlen(str)+1, sizeof(char));
	if (!dst)
		perror("bad alloc");
	char *dstPtr = dst;
	while (*str){
		if (*str != ' ')
			*dstPtr++ = *str;
		str++;
	}
	free(dstPtr);
	return dst;	
}

Возможно, я изобрел велосипед с inputString. Тогда как мне вводить строку с неизвестной длинной?

Программа работает, но valgrind жалуется на утечку памяти:

лог

[user@user CutSpaces]$  valgrind --leak-check=full ./main 
==27117== Memcheck, a memory error detector
==27117== Copyright (C) 2002-2017, and GNU GPL'd, by Julian Seward et al.
==27117== Using Valgrind-3.14.0 and LibVEX; rerun with -h for copyright info
==27117== Command: ./main
==27117== 
sdlfks; dlgsd;gl ksd;gl       
sdlfks;dlgsd;glksd;gl   <-- Оно работает
==27117== 
==27117== HEAP SUMMARY:
==27117==     in use at exit: 24 bytes in 1 blocks
==27117==   total heap usage: 6 allocs, 5 frees, 2,132 bytes allocated
==27117== 
==27117== 24 bytes in 1 blocks are definitely lost in loss record 1 of 1
==27117==    at 0x4839B65: calloc (vg_replace_malloc.c:752)
==27117==    by 0x1092D1: CutSpaces (in /path/to)
==27117==    by 0x1091BA: main (in /path/to)
==27117== 
==27117== LEAK SUMMARY:
==27117==    definitely lost: 24 bytes in 1 blocks
==27117==    indirectly lost: 0 bytes in 0 blocks
==27117==      possibly lost: 0 bytes in 0 blocks
==27117==    still reachable: 0 bytes in 0 blocks
==27117==         suppressed: 0 bytes in 0 blocks
==27117== 
==27117== For counts of detected and suppressed errors, rerun with: -v
==27117== ERROR SUMMARY: 1 errors from 1 contexts (suppressed: 0 from 0)

Укажите на ошибки в коде

Comments

[Roman]

//...
str = calloc(size, sizeof(char));
//...
char *tmp  = realloc(str, sizeof(char)*(size+=16));
//...
str = tmp;
//...

ну и на всякий случай
if (!str) return str;
...
а что если действительно нулевой

str = inputString(10);	
new_str = CutSpaces(str);	
printf("%s\n", new_str);

[Hello World]

Roman, что нулевой?

[Roman]

Hello World,
https://en.cppreference.com/w/c/memory/calloc

Return value
On success, returns the pointer to the beginning of newly allocated memory. To avoid a memory leak, the returned pointer must be deallocated with free() or realloc().
On failure, returns a null pointer.

str = calloc(size, sizeof(char));
if (!str) return str;
...
char *dst = calloc(strlen(str)+1, sizeof(char));

https://onlinegdb.com/rk0wwWfdE

[Hello World]

Roman, я должен делать free(str) при фейле и str=NULL? Спасибо

[Roman]

Hello World, нет, на NULL проверять strlen(str)
Segmentation fault вы уже словили.

Answer 1

[jcmvbkbc]

Укажите на ошибки в коде

Можно собрать программу с -g, и valgrind сам тебе укажет с точностью до строчки в исходниках, где была выделена утёкшая память.

как мне вводить строку с неизвестной длинной?

char *str = NULL;
scanf("%m[^\n]", &str);

Comments

[Hello World]

после scanf для теста написал printf, но он seg fault. почему?

[jcmvbkbc]

почему?

Hello World, дебагер?

[Hello World]

jcmvbkbc,

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

char *CutSpaces(char *str);

int main(){
        char *str=NULL,*new_str;

        scanf("%m[^\n]", &str);
        printf("Old: %s\n",str);
        new_str = CutSpaces(str);
        printf("New: %s\n", new_str);

        free(new_str);
        free(str);

        return 0;
}

char *CutSpaces(char *str){
        char *dst = calloc(strlen(str)+1, sizeof(char));
        if (!dst) perror("bad alloc");
        char *dstPtr = dst;
        while (*str){
                if (*str != ' ')
                        *dstPtr++ = *str;
                str++;
        }

        free(dstPtr);
        return dst;
}

$ gcc cut.c -o cut
$ ./cut
WARNING: linker: ./cut: unused DT entry: type 0x6ffffef5 arg 0x24c
hsd gad
Old: (null)
Segmentation fault

[res2001]

jcmvbkbc, Вроде модификатора 'm' в стандарте Си нет. На сколько помню - это POSIXовая штука.

[jcmvbkbc]

модификатора 'm' в стандарте Си нет

res2001, это верно