Récupérer les couleurs d'une image - Violation d'accès

Version imprimable

1 pièce(s) jointe(s)

Bonjour, je cherche à récupérer les couleurs d'une image afin de la passer en gris.

Pour cela j'utilise une fonction on_click_button_grey qui récupère un builder :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
 
void on_click_button_grey(GtkButton *button, gpointer user_data){
	GtkBuilder *builder = (GtkBuilder*)user_data;
	GdkPixbuf *pixbuf = NULL;
	GtkImage *image1 = NULL;
 
	int w = 0;
	int h = 0;
	int x = 0;
	int y = 0;
	int gris = 0;
 
	guchar *red= NULL;
	guchar *green = NULL;
	guchar *blue = NULL;
	guchar *buffer=NULL;
	gint pos = 0;
	gint row_size = 0;
 
	image1 = GTK_IMAGE(gtk_builder_get_object(builder, "image1"));
 
	pixbuf = gdk_pixbuf_new_from_file("image.png", NULL);
	w = gdk_pixbuf_get_width(pixbuf);
	h = gdk_pixbuf_get_height(pixbuf);
 
	buffer = gdk_pixbuf_get_pixels(pixbuf);
	row_size = gdk_pixbuf_get_rowstride(pixbuf);
 
	for (y = 0; y < h; y++) {
		for (x = 0; x < w; x++) {
 
			if (gdk_pixbuf_get_has_alpha(pixbuf))
				pos = (y * row_size) + (x * 4);
			else
				pos = (y * row_size) + (x * 3);
 
			*red = buffer[pos];
			*green = buffer[pos + 1];
			*blue = buffer[pos + 2];
 
			gris = (int)(*red*0.3 + *green*0.59 + *blue*0.11);
 
			buffer[pos] = gris;
			buffer[pos + 1] = gris;
			buffer[pos + 2] = gris;
 
		}
	}
 
	gtk_image_set_from_pixbuf(image1, pixbuf);
	g_object_unref(pixbuf);
 
	return;
}

Malheureusement lors de l'exécution du programme j'ai une erreur de violation d'accés que je ne comprends pas en ligne 37, c'est à dire au moment où je veux renseigner *red.

Pièce jointe 498151

Merci d'avance pour votre aide ! :P

19/08/2019, 09h25
gerald3d
Bonjour.

Lorsque tu manipules des pointeurs il est fortement conseillé de toujours vérifier leur validité.

Au vu de la réponse de ton compilateur, il semblerait que buffer pointe sur l'adresse 0x0000, c'est à dire la valeur NULL !

Si on reprend la documentation de gdk_pixbuf_new_from_file (); cette fonction peut renvoyer NULL si elle a rencontré des problèmes.

Citation:

Returns

A newly-created pixbuf with a reference count of 1, or NULL if any of several error conditions occurred: the file could not be opened, there was no loader for the file's format, there was not enough memory to allocate the image buffer, or the image file contained invalid data.

Donc ton code devrait ressembler à ceci :
Code:

1 2 3 4 ... pixbuf = gdk_pixbuf_new_from_file("image.png", NULL); g_return_if_fail (pixbuf); ...
Cette fonction évalue l'expression entre parenthèses. S'il est fausse on sort. Ceci équivaut à :

Code:

if (pixbuf==NULL) return;

À partir de là, si l'exécution du code se poursuit tu sais que ton pixbuf est correctement initialisé.

Autre petite remarque. Pourquoi initialiser des pointeurs pour les couleurs ? De simples variables suffisent ici il me semble. Ca simplifierait aussi l’écriture du code source.
19/08/2019, 10h10
Christophe

Je fais pas de programmation, je vais donc peut-être dire une bêtise, tu appelles un pointeur red en ligne 37 sur laquelle tu n'as pas fait de malloc avant.
19/08/2019, 16h15
liberforce
Pour gérer correctement l'erreur, il faut la récupérer.

Code:

pixbuf = gdk_pixbuf_new_from_file("image.png", NULL);

Le second argument, où tu passes NULL, est en fait l'adresse d'un pointeur où récupérer des informations sur l'erreur en utilisant GError.
Code:

1 2 3 4 5 GError *error = NULL; pixbuf = gdk_pixbuf_new_from_file("image.png", &error); if (error != NULL) g_error("%s", error->message);
g_error permet de tracer (logger) l'erreur, et comme c'est une erreur fatale, d'arrêter le programme, tout en affichant la cause de l'erreur. Vraisemblablement le fichier image n'est pas trouvé (sans doute parce que tu as mis un chemin relatif mais pas relatif au bon dossier; teste avec un chemin absolu pour voir), ou est mal nommé.