Kinsey Moore started a new discussion on 
rtemsbsd/rtems/rtems-bsd-shell-wpa_supplicant_fork.c: 
https://gitlab.rtems.org/rtems/pkg/rtems-libbsd/-/merge_requests/123#note_143296

 > -    struct myparams *params = malloc(sizeof(struct myparams));
 > +    struct myparams *params = (struct myparams*)malloc(sizeof(struct 
 > myparams));
 >      if (params == NULL)
 > -            return NULL;
 > +            return -1;
 >  
 >      params->argc = argc;
 > -    params->argv = malloc((argc + 1) * sizeof(argv[0]));
 > +    params->argv = (char**)malloc((argc + 1) * sizeof(argv[0]));
 >      if (params->argv == NULL)
 > -            return NULL;
 > +            return -1;
 >  
 >      for (i = 0; i < argc; i++) {
 > -            params->argv[i] = strdup(argv[i]);
 > +            params->argv[i] = (char*)strdup(argv[i]);

`strdup()` should already be returning a `char *`. Why is this necessary?

-- 
View it on GitLab: 
https://gitlab.rtems.org/rtems/pkg/rtems-libbsd/-/merge_requests/123#note_143296
You're receiving this email because of your account on gitlab.rtems.org.


_______________________________________________
bugs mailing list
[email protected]
http://lists.rtems.org/mailman/listinfo/bugs

Reply via email to