Opened 6 years ago
Closed 6 years ago
#1064 closed Bug / Defect (fixed)
IP address displayed in Windows GUI may be incorrect
| Reported by: | firthmj | Owned by: | Selva Nair |
|---|---|---|---|
| Priority: | minor | Milestone: | release 2.4.7 |
| Component: | Windows GUI | Version: | OpenVPN 2.4.5 (Community Ed) |
| Severity: | Not set (select this one, unless your'e a OpenVPN developer) | Keywords: | |
| Cc: |
Description
If you connect and are assigned an IP address with a certain number of digits (e.g. 10.20.100.10), and then reconnect and are assigned an IP address with less digits (e.g. 10.20.100.6), the display will incorrectly show a string with the extra digits from the old address appended - it would show 10.20.100.60 for this example.
Actual addresses don't match my configuration, but the lengths do.
This looks like a fairly trivial "strcpy" or equivalent error somewhere in the code, where it isn't copying / applying the terminator correctly when the string is updated.
The error is cosmetic, but it is confusing
Change History (3)
comment:1 Changed 6 years ago by
| Owner: | changed from Heiko Hund to Selva Nair |
|---|---|
| Status: | new → assigned |
comment:2 Changed 6 years ago by
| Milestone: | → release 2.4.7 |
|---|---|
| Status: | assigned → accepted |
comment:3 Changed 6 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | accepted → closed |
Fixed by PR 261 (https://github.com/OpenVPN/openvpn-gui/pull/261)
Closing.
Note: See
TracTickets for help on using
tickets.
On a quick scan it seems the parsed ip address is not explicitly null-terminated that can indeed cause this behaviour. The whole buffer is cleared at the start and is long enough to never over-fill, so there is no buffer overflow concern. Will fix in next release.