Question

Can anyone tell me what is wrong with the following code?

__inline__
char* ut_byte_to_long (ulint nb) {

   char* a = malloc(sizeof(nb)); 
   int i = 0;
   for (i=0;i<sizeof(nb);i++) {
       a[i] = (nb>>(i*8)) & 0xFF;
   }
   return a; 
}

This string is then concatenated as part of a larger one using strcat. The string prints fine but for the integers which are represented as character symbols. I’m using %s and fprintf to check the result.

Thanks a lot.

EDIT

I took one of the comments below (I was adding the terminating \0 separately, before calling fprintf, but after strcat. Modifying my initial function…

__inline__
char* ut_byte_to_long (ulint nb) {

   char* a = malloc(sizeof(nb) + 1); 
   int i = 0;
   for (i=0;i<sizeof(nb);i++) {
       a[i] = (nb>>(i*8)) & 0xFF;
   }
   a[nb] = '\0' ; 
   return a; 
}

This sample code still isn’t printing out a number…

char* tmp;
tmp = ut_byte_to_long(start->id);

fprintf(stderr, "Value of node is %s \n ", tmp);

Answer 1

if you dont want to use sprintf(target_string,"%lu",source_int) or the non standard itoa(), here is a version of the function that transform a long to a string :

__inline__
char* ut_byte_to_long (ulint nb) {
    char* a = (char*) malloc(22*sizeof(char));  
    int i=21;
    int j;
    do
    {
        i--;
        a[i] = nb % 10 + '0';
        nb = nb/10; 
    }while (nb > 0);
    // the number is stored from a[i] to a[21]

    //shifting the string to a[0] : a[21-i]
    for(j = 0 ; j < 21 && i < 21 ; j++ , i++)
    {
        a[j] = a[i];
    }
    a[j] = '\0';
    return a;
}

I assumed that an unsigned long contain less than 21 digits. (biggest number is 18,446,744,073,709,551,615 which equals 2^64 − 1 : 20 digits)