image convolution from ppm file - c

Question

Good afternoon to everyone.
Firstly, i would like to apologize for my code - i am a real beginner in C. My problem is - i am given ppm file and i would need to store values from there into an array. I have already stored height , width and max value of color, now my idea to store a values would be the as shown at the picture - multiplying by three because it is in R G B format.
thank you for your help and please concider the fact i am a real beginner in C.
my code and output
#include <stdio.h>
int main(int argc, char** argv) {
int i = 0;
int j = 0;
FILE *fp;
fp = fopen(argv[1], "r");
printf(" %s ", argv[1]);
printf("\n");
int firstLine[2];
int width;
int next;
int enter;
int loop;
int height;
int max_color;
int pix[width][height];
int mask[3][3] = {// inicializting our given mask
{0, -1, 0},
{-1, 5, -1},
{0, -1, 0}
};
for (i = 0; i < 3; i++) {
for (j = 0; j < 3; j++) {
printf("%d ", mask[i][j]);
}
printf("\n");
}
fscanf(fp, "%s", &firstLine);
fscanf(fp, "%d", &height);
fscanf(fp, "%d", &width);
fscanf(fp, "%d", &max_color);
printf("%p", firstLine);
printf("\n");
printf("%d ", width);
printf("\n");
printf("%d", height);
printf("\n");
printf("%d", max_color);
printf("\n");
for (i = 0; i < width * 3; i++) {
for (j = 0; j < height * 3; j++) {
loop = fscanf(fp, "%d", &enter);
pix[i][j] = enter;
printf("%d ", enter);
}
}
// fclose(fp);
return (EXIT_SUCCESS);
}

Answer 1

You cannot define array with unknown size, esp height and width are not initialized. You should use dynamic allocated array here, like this:
int ***pix;
pix = malloc(height * sizeof(int**));
for (i = 0; i < height; i++) {
pix[i] = malloc(width * sizeof(int**));
for (j = 0; j < width; j++) {
pix[i][j] = malloc(3 * sizeof(int));
}
}
To correct parse the binary, you cannot use formatted input because they are for strings. You can use fread instead, e.g. to read the width:
fread(&width, sizeof(int), 1, fp);
Then to fill this array:
for (i = 0; i < height; i++) {
for (j = 0; j < width; j++) {
for (k = 0; k < 3; k++) {
fread(&enter, sizeof(int), 1, fp);
pix[i][j][k] = enter;
}
}
}
This only works if what you said about ppm file's format is correct of course.