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

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

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

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

Q&A

解決済

3回答

798閲覧

calloc unsigned charでのメモリ確保ができない

Lizard_knight

総合スコア18

C

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

0グッド

0クリップ

投稿2019/07/12 07:37

calloc関数でunsigned charでのメモリ確保を実現したい

bainaryファイルを読み込み、(622,25088) の配列に格納するプログラムを書いています。
配列を生成するときに、動的メモリ確保する関数callocを使っていますが、その際にエラーが出ます。

発生している問題・エラー

… now frame 148 now frame 149 now frame 150 now frame 151 now frame 152 now frame 153 now frame 154 now frame 155 Segmentation fault (core dumped)

該当のソースコード

c/c++

1#include <iostream> 2#include <stdio.h> 3#include <stdlib.h> 4#include <string.h> 5 6//#define debug 7 8int main(void){ 9 char inname[100] = "./input.txt"; 10 FILE *infp; 11 infp=fopen(inname,"rb"); 12 if(infp==NULL){ 13 perror("input file open error"); 14 exit(-1); 15 } 16 17 int **matrix, *base_matrix; 18 int frame=622, channel=512, winsize=7; 19 int tmp=channel*winsize*winsize; 20 unsigned char buf; 21 22 matrix=(int**)calloc(sizeof(int *), frame); 23 base_matrix=(int*)calloc(sizeof(unsigned char), frame*tmp); 24 for(int i=0;i<frame;i++){ 25 matrix[i] = base_matrix+i*tmp; 26 } 27 28 //printf("test :%d\n",tmp); //ok 29 30 for(int i=0;i<frame;i++){ 31 printf("now frame %d\n", i); 32 for(int j=0;j<tmp;j++){ 33 //if(i==155){ 34 // printf("j:%d\n",j); 35 //} 36 fread(&buf,1,1,infp); 37 matrix[i][j]=buf; 38#ifdef debug 39 if((j+1)%49==0){ 40 printf("%3d\n\n", matrix[i][j]); 41 }else if((j+1)%7==0){ 42 printf("%3d\n", matrix[i][j]); 43 }else{ 44 printf("%3d ", matrix[i][j]); 45 } 46#endif 47 } 48 } 49 50 return 0; 51} 52

試したこと

base_matrix=(int*)calloc(sizeof(unsigned char), frame*tmp);
の行の「unsigned char」を「int」に変更するとセグフォがなくなり動きます。
bufを代入しているので、unsigned charでのメモリ確保でできるはずなのですが...

補足情報(FW/ツールのバージョンなど)

g++ (Ubuntu 5.4.0-6ubuntu1~16.04.10) 5.4.0 20160609

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

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

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

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

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

guest

回答3

0

ベストアンサー

1バイトずつ扱いたいのであれば、matrixunsigned char **として、base_matrixunsigned char *として宣言する必要があるかと思います。

int *では1つ値を動かすとintのサイズだけポインタが動きますので、frame*tmpバイトのメモリでは、matrix[frame-1][tmp-1]でアクセスさせるのに必要なメモリには全く足りません。

投稿2019/07/12 07:43

maisumakun

総合スコア145184

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

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

Lizard_knight

2019/07/12 07:50

出来ました!ありがとうございます!!
guest

0

c

1 int **matrix, *base_matrix; 2 base_matrix=(int*)calloc(sizeof(unsigned char), frame*tmp); 3 for(int i=0;i<frame;i++){ 4 matrix[i] = base_matrix+i*tmp; 5 }

i=frame-1のとき
matrix[i] = matrix[frame-1] = base_matrix+(frame-1)*tmp
になります。

問題になるのが、intポインタ(int*)に数値を加算した場合

c

1unsigned char p: 2int* ptr = p; 3ptr+1 == p+(sizeof(int))*1

になることです。

つまり、base_matrix+(frame-1)*tmpsizeof(int)*(frame-1)*tmpだけbase_matrixより大きいアドレスになります。

投稿2019/07/12 07:52

asm

総合スコア15147

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

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

0

mem1 = calloc(sizeof(unsigned char), 100); mem2 = calloc(sizeof(int), 100);

mem1には100byteのメモリが、mem2には400byteのメモリが確保されています
この違いを理解してないんだと思います

投稿2019/07/12 07:43

izmktr

総合スコア2856

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

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

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.48%

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

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

質問する

関連した質問