This is the mail archive of the ecos-bugs@sourceware.org mailing list for the eCos project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[Bug 1001522] Array index out of bounds in tftp_server.c


Please do not reply to this email. Use the web interface provided at:
http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001522

--- Comment #4 from Grant Edwards <grant.b.edwards@gmail.com> 2012-03-09 16:11:42 GMT ---
(In reply to comment #3)

> Why not move the "close loop" to below the switch statement that calls
> tftpd_send_error()?

Because then the sockets don't get closed until after the file
operation completes and you loose the possibility of doing multiple
file operations in parallel?

If that's the case, then the ckeck for invalid opcode and error
response needs to happen before the "close loop", but the actual
handling of the read/write opcode needs to happen after the "close
loop"?

I must admit, I don't really understand why the sockets are being
closed at all.  Can't multiple threads read from a single socket?

-- 
Configure bugmail: http://bugs.ecos.sourceware.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]