Warning compilation conversion

**vodkline** · 09/02/2017, 17h51

Bonjour,
je suis entrain de déveloper un programme et j'ai un warning

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
attention : assignment makes pointer from integer without a cast [-Wint-conversion]
   if ((tmp = sbrk(size)) == NULL)

voila mon code :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
 Unit                *g_unit= NULL;
 
 
 int                    getMemory()
 {
   Unit              *tmp;
 
   size = getSize();
   if ((tmp = sbrk(size)) == NULL)
     return 0;
   printf("%x", tmp);
  return (0);
}

je ne comprend pas mon erreur, pouvez vous m'aider ? merci d'avance

**dalfab** · 09/02/2017, 18h53

Bonjour,

L'include indiquant bien que sbrk() retourne un void* a-t-il été fait? (<unistd.h> il me semble)

**vodkline** · 09/02/2017, 21h07

Bonjour,
l'include a bien étais inclue dans le projet.
Il trouve bien le prototype.

**gangsoleil** · 10/02/2017, 17h15

Bonjour,

size est-il du bon type ?

**vodkline** · 11/02/2017, 00h02

oui size est de type size_t

**ternel** · 13/02/2017, 09h26

"assignement"
le seul "assignement", c'est unit = sbrk(...). Que retourne sbrk()? je parierai sur un size_t (donc un entier), plutot que void*.

**nnovic** · 13/02/2017, 14h17

Salut,

sbrk() semble, sur certains systèmes, attendre un pointeur comme argument. Quel est le type de 'size' ?

**sambia39** · 13/02/2017, 20h12

Bonsoir,

Envoyé par vodkline

Bonjour,
je suis entrain de déveloper un programme et j'ai un warning

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
attention : assignment makes pointer from integer without a cast [-Wint-conversion]
   if ((tmp = sbrk(size)) == NULL)

je ne comprend pas mon erreur, pouvez vous m'aider ? merci d'avance

C'est tout à fait normale que le compilateur émet des warnings et ceux pour des raisons suivantes.
La première est que sur certain système d'exploitation comme Mac Os ou UNIX; émettent tous deux des warnings en précisant que "sbrk is deprecated ~~void *sbrk(int)~~" deuxième points le warning vient également du fait que vous avez effectué une conversion de pointeur en un entier sans utilisation de l'opérateur de conversion.

Comme sur certain système d'exploitation l'utilisation de "sbrk" n'est pas recommandée il faut dans ce cas (si je peux le dire ainsi) contourné le problème en utilisant le mots-clés "extern" en redéfinisant la fonction et ainsi permettre au compilateur de connaitre/importer la fonctions "adéquat" définies dans le fichier source "unistd h". Sur GNU/Linux pas besoin de externe sauf si peut-être dans vos option de compilation vous exigez plus. Tout de même sur Unix et Mac Os le warning restera d'actualité sauf si vous ignorez le warning.
Voici un exemple: Attention le code-souces ci-dessous est susceptible de comporter des erreurs.
-> Cas 1 extern sbrk mise sous commentaire.

Code C :

Sélectionner tout - Visualiser dans une fenêtre à part

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
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
 
/*
 ============================================================================
 Name        : source.c
 Date        : 13/02/2017
 Author      : SAMBIA39
 Version     : 0.1
 Copyright   : Copyright © 2017 SAMBIA39 All rights reserved.
 Description : Ansi-style
 ============================================================================
 */
 
#define MAXSIZE_UINT 10		/* Taille max d'une variable unsigned int  */
#define MAXSIZE_CHAR 50		/* Taille max d'une variable unsigned char */
 
#include <errno.h>
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <stdint.h>
#include <unistd.h>
 
 
/*
*	La fonction utilise sbrk qui permet d'incrémenté 
*	l’espace de données du programme.
*      Nul permet d’obtenir l’emplacement de la limite actuelle.
*      le contraire a pour effet d’allouer de la mémoire au processus.
*	la reduire à  pour  effet de désallouer de la mémoire.
*
*	/!\ Attetion sur certain système la fonction est "deprecated"
*/
void *f_get_sbrk_mem_init( intptr_t size ){
 
	void *p = NULL;
	extern void *sbrk(intptr_t);
	if( ((void*)-1) == ( p = sbrk(size) ) )
		return ( (void*)-1 );
	memset( p, 0x00, size );
	return ( p );
}
 
/*
*	Fonction d'informations
*/
void f_print_addr( const char*p_str, void *pData ){
	fprintf( stdout, "(%s)\t:%p\n",p_str, pData );
}
 
int main( void ){
 
	unsigned int i = 0;
	unsigned int *ptr_uint = NULL;
 
	/*
	* Attention il faut être attentif a la taille 
	* des élemnts que l'on souhaites. Cas contraire
	* sela résultera par des erreurs.
	*/
	ptr_uint = f_get_sbrk_mem_init( 
		MAXSIZE_UINT*sizeof(unsigned int) ); 
	if(	((void*)-1) == (void*)ptr_uint )
		return EXIT_FAILURE;
 
	f_print_addr( "Var ptr_uint", (void*)ptr_uint );
 
	/*
	*	Initialisation & affichage 
	*/
	for( i = 0; i < MAXSIZE_UINT; i++ ){
		*(ptr_uint+i) = i;
		fprintf( stdout, "(%d)\t:%d\n", i,
			*(ptr_uint+i) );
	}
 
	return (EXIT_SUCCESS);
}

Code Sortie :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
root@vm:~$ gcc -std=c99 -Wall -pedantic -Werror -g *.c -o Deb
source.c: In function ‘f_get_sbrk_mem_init’:
source.c:36:27: error: implicit declaration of function ‘sbrk’ [-Werror=implicit-function-declaration]
  if( ((void*)-1) == ( p = sbrk(size) ) )
                           ^~~~
source.c:36:25: error: assignment makes pointer from integer without a cast [-Werror=int-conversion]
  if( ((void*)-1) == ( p = sbrk(size) ) )
                         ^
cc1: all warnings being treated as errors
root@vm

Cas deux

Code c :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
 
/*
*	La fonction utilise sbrk qui permet d'incrémenté 
*	l’espace de données du programme.
*   Nul permet d’obtenir l’emplacement de la limite actuelle.
*   le contraire a pour effet d’allouer de la mémoire au processus.
*	la reduire à  pour  effet de désallouer de la mémoire.
*
*	/!\ Attetion sur certain système la fonction est "deprecated"
*/
void *f_get_sbrk_mem_init( intptr_t size ){
 
	void *p = NULL;
	extern void *sbrk(intptr_t);
	if( ((void*)-1) == ( p = sbrk(size) ) )
		return ( (void*)-1 );
	memset( p, 0x00, size );
	return ( p );
}

Code Sortie :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
 
root@vm:~$ gcc -std=c99 -Wall -pedantic -Werror -g *.c -o Deb
root@vm:~$ ./Deb 
(Var ptr_uint)	:0x8a8f000
(0)	:0
(1)	:1
(2)	:2
(3)	:3
(4)	:4
(5)	:5
(6)	:6
(7)	:7
(8)	:8
(9)	:9
root@vm:~$

Tout de même, si l'on souhaite allouer du mémoire l'idéal reste la fonction d'allocation "malloc". "sbrk" est à utiliser dans des cas bien précis et justifier de plus, quand vous sollicitez de la mémoire, il faudrait également rendre ce qui vous a été prêté. Cependant, dans quels buts utilisiez-vous "sbrk" ?
à bientôt.

**Matt_Houston** · 13/02/2017, 22h48

@sambia : https://www.developpez.net/forums/d1...lisation-sbrk/

Warning compilation conversion

C

Discussions similaires

Partager

Partager