Skip to main content
Code Review

Return to Answer

deleted 31 characters in body
Source Link
Jamal
Jamal
  • 35.2k
  • 13
  • 134
  • 238

You pass &tmp_input&tmp_input to select but then pass &input&input to the FD_ISSETsFD_ISSETs. These two fd_setfd_set objects are different. The input fd_setfd_set is set once, so FD_ISSETFD_ISSET may well lie to you and your reads will fail. I am surprised no one saw this.

stuart

You pass &tmp_input to select but then pass &input to the FD_ISSETs. These two fd_set objects are different. The input fd_set is set once, so FD_ISSET may well lie to you and your reads will fail. I am surprised no one saw this.

stuart

You pass &tmp_input to select but then pass &input to the FD_ISSETs. These two fd_set objects are different. The input fd_set is set once, so FD_ISSET may well lie to you and your reads will fail.

Source Link
stuart
stuart
  • 1

You pass &tmp_input to select but then pass &input to the FD_ISSETs. These two fd_set objects are different. The input fd_set is set once, so FD_ISSET may well lie to you and your reads will fail. I am surprised no one saw this.

stuart