Amar Takhar commented on a discussion on cpukit/ftpd/ftpd.c: 
https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/173#note_110760

 >    struct timeval tv;
 >    int len = sizeof(tv);
 >  
 > +  if(s < 0)

No I mean keeping the code but adding a comment that it was added due to a 
Coverity highlight about -1 possibly being passed in.  To me at least it seems 
like an out of place check I'd want to know why it's there.

-- 
View it on GitLab: 
https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/173#note_110760
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

Reply via email to