質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.48%
C

C言語は、1972年にAT&Tベル研究所の、デニス・リッチーが主体となって作成したプログラミング言語です。 B言語の後継言語として開発されたことからC言語と命名。そのため、表記法などはB言語やALGOLに近いとされています。 Cの拡張版であるC++言語とともに、現在世界中でもっとも普及されているプログラミング言語です。

ポインタ

ポインタはアドレスを用いてメモリに格納された値を"参照する"変数です。

配列

配列は、各データの要素(値または変数)が連続的に並べられたデータ構造です。各配列は添え字(INDEX)で識別されています。

Q&A

解決済

1回答

3663閲覧

C言語 mallocをfreeで解放する際に発生するエラーについて

makkasa

総合スコア11

C

C言語は、1972年にAT&Tベル研究所の、デニス・リッチーが主体となって作成したプログラミング言語です。 B言語の後継言語として開発されたことからC言語と命名。そのため、表記法などはB言語やALGOLに近いとされています。 Cの拡張版であるC++言語とともに、現在世界中でもっとも普及されているプログラミング言語です。

ポインタ

ポインタはアドレスを用いてメモリに格納された値を"参照する"変数です。

配列

配列は、各データの要素(値または変数)が連続的に並べられたデータ構造です。各配列は添え字(INDEX)で識別されています。

0グッド

0クリップ

投稿2020/02/24 02:18

実装したいこととしては、ランダムで発生させた1~100の数値を偶数と奇数に分け、それぞれ配列に代入して出力を試みています。
その際に、偶数・奇数それぞれの数分のメモリ容量をmallocにて確保しています。

しかしながら、実行コードの下部 「ここが無いとエラーが発生」 のコードを入力しないと、
下部エラー画面のようなエラーが発生致します。

free関数で解放する際に、malloc関数で確保した最初の要素のアドレスに戻しておかないと、適切に解放出来ないのでしょうか?

どなたか、原因をご教授いただけますと幸いです。

↓実行コード

#include <stdio.h> #include <time.h> #include <stdlib.h> int main(){ int n; srand((unsigned)time(NULL)); printf("いくつの数値を計算しますか?:"); scanf("%d",&n); int a[n]; int i=0,g=0,k=0; int *pg, *pk; while(i<n){ a[i] = rand() % 100 + 1; if(a[i] % 2 == 0){ g++; }else{ k++; } i++; } pg = (int*)malloc(sizeof(int) * g); pk = (int*)malloc(sizeof(int) * k); for(i=0;i<n;i++){ if(a[i] % 2 == 0){ printf("%d ",a[i]); *pg = a[i]; pg++; }else{ printf("%d ",a[i]); *pk = a[i]; pk++; } } pg = pg - g; pk = pk - k; printf("\n偶数:"); for(i=0;i<g;i++){ printf("%d ",*pg); pg++; } printf("\n奇数:"); for(i=0;i<k;i++){ printf("%d ",*pk); pk++; } printf("\n"); pg = pg - g; //ここが無いとエラー発生 pk = pk - k; //ここが無いとエラー発生 free(pg); free(pk); }

↓エラー画面

pointer being freed was not allocated *** set a breakpoint in malloc_error_break to debug Abort trap: 6

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答1

0

ベストアンサー

free関数で解放する際に、malloc関数で確保した最初の要素のアドレスに戻しておかないと、適切に解放出来ないのでしょうか?

はい、そのとおりです。

free() 関数はポインター ptr が指すメモリー空間を解放する。このポインターは、 以前に呼び出された malloc(), calloc(), realloc() のいずれかが返した値で なければならない。これ以外のポインターを指定したり、すでに free(ptr) が実行 されていたりした場合の動作は定義されていない。(manより)

投稿2020/02/24 02:22

maisumakun

総合スコア145183

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

makkasa

2020/02/24 02:27

maisumakun様 ご丁寧にご解説して頂き、誠にありがとうございます。 勉強になりました。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.48%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問