lwIP - A Lightweight TCP/IP stack - Bugs: bug #50534, TFTP server does not copy...
You are not allowed to post comments on this tracker with your current authentication level.
bug #50534: TFTP server does not copy terminating null of filename
Submitter: | David Rodgers <drodgers_ls> | ||
Submitted: | Mon 13 Mar 2017 08:04:49 PM UTC | ||
Category: | apps | Severity: | 3 - Normal |
Item Group: | Faulty Behaviour | Status: | Fixed |
Privacy: | Public | Assigned to: | None |
Open/Closed: | Closed | Planned Release: | None |
lwIP version: | 2.0.2 |
Mon 13 Mar 2017 08:42:46 PM UTC, comment #2: |
Simon Goldschmidt <goldsimon> |
Mon 13 Mar 2017 08:27:16 PM UTC, comment #1: Thank you for reporting and providing a patch!! Pushed. |
Dirk Ziegelmeier <dziegel> |
Mon 13 Mar 2017 08:04:49 PM UTC, original submission:
I am currently developing a TCP and UDP application on NXP Kinetis using lwIP 2.0.0. NXP does not provide any of the apps, so I copied lwip/src/apps/tftp/tftp_server.c into my application. Everything seemed to work OK, except after some extended testing, I noticed I was sometimes getting trailing garbage in my filenames:
|
David Rodgers <drodgers_ls> |
Depends on the following items: None found
Items that depend on this one: None found
There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.
Follow 3 latest changes.
Date | Changed by | Updated Field | Previous Value | => | Replaced by |
---|---|---|---|---|---|
2017-03-13 | dziegel | Status | None | Fixed | |
Open/Closed | Open | Closed | |||
2017-03-13 | drodgers_ls | Attached File | - | Added tftp_server.c, #40000 |
Powered by Savane 3.13-02a9.
Corresponding source code
In general, it's better to provide a diff/patch against a defined version. This helps us to see what's changed without manually comparing (where we need the version you have for manual diff).
On top of that, a git diff save us the time for a commit (and keeps your name in the git log), but for such simple patches, a simple diff is probably enough :-)