RTEMS | cpukit/libblock: Fixes off-by-1 and memory in flashdisk (!912)
Chris Johns (@chris)
gitlab at rtems.org
Fri Jan 30 00:19:09 UTC 2026
Chris Johns commented on a discussion on cpukit/libblock/src/flashdisk.c: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/912#note_140365
>
> return RTEMS_SUCCESSFUL;
> }
> +
> +static void rtems_fdisk_cleanup_helper (rtems_flashdisk *fd, bool *mutex_initialized)
> +{
> + uint32_t device;
> +
> + if(fd == NULL)
> + return;
> +
> + if((*mutex_initialized) == false) {
> + (*mutex_initialized) = true;
> + } else {
> + rtems_mutex_destroy(&fd->lock);
This mutex destroy call returns a `void` so it would appear to be safe to call is the mutex is initialized or not. If this is the case then need to add a `bool` to track the mutex adds nothing to this change.
--
View it on GitLab: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/912#note_140365
You're receiving this email because of your account on gitlab.rtems.org.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rtems.org/pipermail/bugs/attachments/20260130/55530c99/attachment-0001.htm>
More information about the bugs
mailing list