Kinsey Moore started a new discussion on freebsd/sys/kern/kern_event.c: 
https://gitlab.rtems.org/rtems/pkg/rtems-libbsd/-/merge_requests/71#note_122899

 >                      if (ffd < 0)
 >                              error = EBADF;
 >                      else {
 > -                            if (iop == NULL)
 > +                            if (iop == NULL) {
 >                                      error = fget(td, ffd, 
 > &cap_event_rights, &fp);
 > -                            else
 > +                                    if (!error) {
 > +                                            
 > rtems_bsd_libio_iop_drop(kev->ident);
 > +                                    }

Doesn't this logic need to be replicated in the other locations that use 
rtems_bsd_libio_iop_hold()?

Why not integrate this functionality into rtems_bsd_libio_iop_hold() directly?

-- 
View it on GitLab: 
https://gitlab.rtems.org/rtems/pkg/rtems-libbsd/-/merge_requests/71#note_122899
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