Gedare Bloom commented on a discussion on cpukit/libblock/src/flashdisk.c: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/912#note_139518 > + return; > + > + uint32_t device; > + > + if(fd->devices) > + { > + for(device = 0; device< fd->device_count; device++) > + { > + free(fd->devices[device].segments); > + fd->devices[device].segments = NULL; > + } > + } > + free(fd->devices); > + fd->devices = NULL; > + > + free(fd->blocks); Please don't resolve the threads yourself, leave them open for the reviewer to resolve. You don't need to check for NULL, but you do have to be certain that everything is initialized to NULL **before** calling `calloc`, because this function gets called without completing the `rtems_fdisk_initialize`. Since the `fd` is created with `calloc` then this should be the case, so it's fine. -- View it on GitLab: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/912#note_139518 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
