MITHILESH MATTAPALLI commented on a discussion on cpukit/libblock/src/flashdisk.c: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/903#note_145331 > + free( p->devices ); > + } > + free( p->blocks ); > + free( p->copy_buffer ); > + } > + > + if ( fd ) { > + uint32_t d; > + > + if ( fd->devices ) { > + for ( d = 0; d < fd->device_count; d++ ) { > + free( fd->devices[ d ].segments ); > + } > + free( fd->devices ); > + } > + free( fd->blocks ); Since `fd` is just pointing directly to an index in the allocated array now, it can't be NULL.So, I went ahead and dropped the check -- View it on GitLab: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/903#note_145331 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
