Warning compilation conversion

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

Code:

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

voila mon code :

Code:

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

Bonjour,

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

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

Bonjour,

size est-il du bon type ?

oui size est de type size_t

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

Salut,

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

Bonsoir,
Citation:
Envoyé par vodkline
Bonjour,
je suis entrain de déveloper un programme et j'ai un warning
Code:

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:

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:

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:

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:

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.

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