Assignment makes integer from pointer without a cast как исправить

/* initial declaration */
char *src = "abcdefghijklmnop";
.....

/* I get the   "warning: assignment makes integer from pointer without a cast" */
*src ="anotherstring";

1

Код: Выделить всё

#define MAXLINE 100
char* get_name_addition(void)
{
    static char name[MAXLINE];
    fgets(name,MAXLINE,stdin);
    ... // Какие-то дополнительные действия
    return *name;
}

Код: Выделить всё

#define MAXLINE 100
int get_name_addition(char *name, size_t n)
{
    fgets(name, n, stdin);
    ...
    return 0;
}

Код: Выделить всё

#define MAXLINE 100
char* get_name_addition(void)
{
    char* name;
    name = (char*)malloc(MAXLINE);
    fgets(name, MAXLINE, stdin);
    ...
    return name;
}

void get_name_free(char* name)
{
    free(name);
}

Код: Выделить всё

int main(void)
{
    char *name;
    name = get_name_addition();
    // Что-то делаем
    ...

    get_name_free(name);

    return 0;
}

• 
• Forum
• The Ubuntu Forum Community
• Ubuntu Specialised Support
• Development & Programming
• Programming Talk
• [SOLVED] C — assigment makes integer from pointer without a cast warning

1. [SOLVED] C — assigment makes integer from pointer without a cast warning
   

   I know it’s a common error, and I’ve tried googling it, looked at a bunch of answers, but I still don’t really get what to do in this situation….
   Here’s the relevant code:

   Code:

   #include <stdio.h>
   #include <stdlib.h>
   #include <string.h>
   
   int main( int argc, char *argv[] )
   {
       char path[51];
       const char* home = getenv( "HOME" );
   
       strcpy( path, argv[1] );
   
       path[1] = home;
   
       return 0;
   }

   — there is more code in the blank lines, but the issue’s not there (I’m fairly sure), so didn’t see the point in writing out 100 odd lines of code.
   I’ve tried some stuff like trying to make a pointer to path[1], and make that = home, but haven’t managed to make that work (although maybe that’s just me doing it wrong as opposed to wrong idea?)

   Thanks in advance for any help

   ---

2. Re: C — assigment makes integer from pointer without a cast warning
   

   path[1] is the second element of a char array, so it’s a char. home is a char *, i.e. a pointer to a char.

   You get the warning because you try to assign a char* to a char.

   Note also the potential to overflow your buffer if the content of the argv[1] argument is very long. It’s usually better to use strncpy.

   Last edited by r-senior; March 10th, 2013 at 03:03 PM.

   Reason: argv[1] would overflow, not HOME. Corrected to avoid confusion.

   Please create new threads for new questions.
   Please wrap code in code tags using the ‘#’ button or enter it in your post like this: [code]…[/code].

   ---

3. Re: C — assigment makes integer from pointer without a cast warning
   

   You can try something like this:

   Code:

   #include <stdio.h>
   #include <stdlib.h>
   #include <string.h>
   
   int main( int argc, char *argv[] )
   {
     char *path;
     const char *home = getenv("HOME");
   
     path = malloc(strlen(home) + 1);
     if (!path) {
       printf("Errorn");
       return 0;
     }
     strcpy(path, home);
     printf("path = %sn", path);
   
     // if you want to have argv[1] concatenated with path
     if (argc >= 2) {
       path = malloc(strlen(home) + strlen(argv[1]) + 1);
       strcpy(path, argv[1]);
       strcat(path, home);
       printf("%sn", path);
     }
   
     // if you want an array of strings, each containing path, argv[1]...
     char **array;
     int i;
   
     array = malloc(argc * sizeof(char*));
   
     array[0] = malloc(strlen(home) + 1);
     strcpy(array[0], home);
     printf("array[0] = %sn", array[0]);
   
     for (i = 1; i < argc; i++) {
       array[i] = malloc(strlen(argv[i]) + 1);
       strcpy(array[i], argv[i]);
       printf("array[%d] = %sn", i, array[i]);
     }
     // now array[i] will hold path and all the argv strings
   
     return 0;
   }

   Just as above, your path[51] is a string while path[1] is only a character, so you can’t use strcpy for that.

   Last edited by Christmas; March 10th, 2013 at 09:51 PM.

   ---

4. Re: C — assigment makes integer from pointer without a cast warning
   

   Originally Posted by Christmas
   

   You can try something like this:

   Code:

   #include <stdio.h>
   #include <stdlib.h>
   #include <string.h>
   
   int main( int argc, char *argv[] )
   {
     char *path;
     const char *home = getenv("HOME");
   
     path = malloc(strlen(home) + 1);
     if (!path) {
       printf("Errorn");
       return 0;
     }
     strcpy(path, home);
     printf("path = %sn", path);
   
     // if you want to have argv[1] concatenated with path
     if (argc >= 2) {
       path = malloc(strlen(home) + strlen(argv[1]) + 1);
       strcpy(path, argv[1]);
       strcat(path, home);
       printf("%sn", path);
     }
   
     // if you want an array of strings, each containing path, argv[1]...
     char **array;
     int i;
   
     array = malloc(argc * sizeof(char*));
   
     array[0] = malloc(strlen(home) + 1);
     strcpy(array[0], home);
     printf("array[0] = %sn", array[0]);
   
     for (i = 1; i < argc; i++) {
       array[i] = malloc(strlen(argv[i]) + 1);
       strcpy(array[i], argv[i]);
       printf("array[%d] = %sn", i, array[i]);
     }
     // now array[i] will hold path and all the argv strings
   
     return 0;
   }

   Just as above, your path[51] is a string while path[1] is only a character, so you can’t use strcpy for that.

   Excellent point. I’ve basically fixed my problem by reading up on pointers again (haven’t done any C for a little while, so forgot some stuff), and doing:
   the code doesn’t moan at me when I compile it, and it runs okay (for paths which aren’t close to 51 at least), but after reading what you read, I just wrote a quick program and found out that getenv(«HOME») is 10 characters long, not 1 like I seem to have assumed, so I’ll modify my code to fix that.

   ---

5. Re: C — assigment makes integer from pointer without a cast warning
   

   Yes, getenv will return the path to your home dir, for example /home/user, but path[1] = *home will still assign the first character of home to path[1] (which would be ‘/’).

   ---

Tags for this Thread