Merge request https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/462 
was reviewed by Joel Sherrill

--
  
Joel Sherrill started a new discussion on testsuites/psxtests/psxaio05/init.c: 
https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/462#note_122323

 >  
 > -  status = lio_listio( LIO_NOWAIT, aiocbp, 4, &sigev);
 > +  status = lio_listio( LIO_NOWAIT, aiocbp, AIO_LISTIO_MAX, &sigev);

This doesn't look right to me. aiocbp points to an array that is of length 4. 
The 4 as the argument was correct. I am not sure how this didn't end up 
referencing bad data since it should have been processing 16 entries that were 
not there.

--
  
Joel Sherrill started a new discussion on testsuites/psxtests/psxaio05/init.c: 
https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/462#note_122324

 >  
 > -  status = lio_listio( LIO_WAIT, aiocbp, 4, NULL);
 > +  status = lio_listio( LIO_WAIT, aiocbp, AIO_LISTIO_MAX, NULL);

Same comment.


-- 
View it on GitLab: 
https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/462
You're receiving this email because of your account on gitlab.rtems.org.


_______________________________________________
bugs mailing list
bugs@rtems.org
http://lists.rtems.org/mailman/listinfo/bugs

Reply via email to