RTEMS | cpukit: Correct comparisons of different signedness (!826)

Kinsey Moore (@opticron) gitlab at rtems.org
Thu Nov 6 17:16:42 UTC 2025




Kinsey Moore started a new discussion on cpukit/dev/flash/flashdev.c: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/826#note_135751

 >    rtems_flashdev_region *region_in
 >  )
 >  {
 > -  uint32_t region_index = rtems_flashdev_get_region_index( iop );
 > +  int region_index = rtems_flashdev_get_region_index( iop );

The more correct fix here is likely to be changing max_regions in flashdev.h to uint32_t since max_regions should never be negative.

-- 
View it on GitLab: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/826#note_135751
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/20251106/188b1307/attachment.htm>


More information about the bugs mailing list