このプログラムはメモリリークを起こしていますか？

前提・実現したいこと

このプログラムはメモリリークを起こしているでしょうか？また、プログラム的に間違っていますか？

該当のソースコード

c
1#include <stdio.h>
2#include <stdlib.h>
3#include <string.h>
4int main()
5{
6    char *x;
7    char a[100] = "panda";
8    char b[100] = "abc";
9    x = (char *)malloc(strlen(a) + 1);
10    strcpy(x, a);
11    x = (char *)malloc(strlen(b) + 1);
12    strcpy(x, b);
13    free(x);
14}

行動規範の内容に同意します

回答1件

ベストアンサー

このプログラムはメモリリークを起こしているでしょうか？

「はい」とも「いいえ」とも言えます。
freeせずに新しいメモリを割り当てポインタ変数に上書きした時点で元のメモリを開放する手段が失われ
メモリリークしている状態になります。

しかし、多くのOSではプロセス終了時に割り当てたメモリは開放されますのでmain関数終了後にメモリリークは解消されます。

また、プログラム的に間違っていますか？

プロセス終了時のメモリ開放を当てにするのはよい習慣とは言えず「行儀が悪い」プログラムとなります。

未定義動作などは踏んでいないので、あなたの意図通りに動いているのであれば「間違ったプログラム」とは言えません。

また、strlen後にstrcpyする事は若干の無駄があるのでstrlenの結果を変数に入れてmemcpyしてもよいでしょう。

投稿2021/07/31 15:48

asm

総合スコア15149

pikahyogo

2021/07/31 16:14

詳しい回答ありがとうございます！ memcpyという関数は使ったことないのですが、この場合、 ```c int num = strlen(b) + 1; x = (char *)malloc(num); memcpy(x,b,num); ``` というように使用すればmemcpyの使用法合っていますか？また、若干の無駄というのは、strcpyだとヌル文字の場所を調べる手間があるので長さを元から求めてるんだったらその手間が無いmemcpyの方が良いという認識で合っていますか？ご回答よろしくお願いします。

episteme

2021/08/01 03:37 編集

> 「はい」とも「いいえ」とも言えます。異議あり...かな。プログラムはメモリ・リークを起こしてて、それをOSが尻ぬぐいしてくれてる(だけ)とするのが真っ当では?

asm

2021/08/01 12:08

> というように使用すればmemcpyの使用法合っていますか？ > だったらその手間が無いmemcpyの方が良いという認識で合っていますか？はい、そうです。 > それをOSが尻ぬぐいしてくれてる(だけ)とするのが真っ当では? OSによってはドキュメントに書かれている仕様ですので利用するのも手段の一つと思います。前述したように行儀が悪いですし、移植性に難があるので避けた方がよいのは確かです。

pikahyogo

2021/08/02 21:52

ご回答ありがとうございました。