| 2022-12-01 08:44:34 |
Tanguy Pelado |
bug |
|
|
added bug |
| 2022-12-01 09:27:30 |
Tanguy Pelado |
description |
Hello all,
We recently encountered a bug in net-snmp V5.9.1 related to DNS name length (https://github.com/net-snmp/net-snmp/issues/476)
Here's a summary :
What I did :
snmpstatus -v2c -c public aaaaaaaaaaaaaaaaaaaaaaaaa.aaaaaaaaaaaaaaaaaaaaaaaaa.taledo.games
What I expected to happen :
Get a timeout from this domain because there's no snmpd running, and I wouldn't use public as a community anyway.
What I got :
snmpstatus: Unknown host (aaaaaaaaaaaaaaaaaaaaaaaaa.aaaaaaaaaaaaaaaaaaaaaaaaa.taledo.games) (No such file or directory)
This domain does exist, and does ping properly (also has V4, this is not the problem here):
PING aaaaaaaaaaaaaaaaaaaaaaaaa.aaaaaaaaaaaaaaaaaaaaaaaaa.taledo.games(2001:41d0:801:2000::539f) 56 data bytes
64 bytes from 2001:41d0:801:2000::539f: icmp_seq=1 ttl=46 time=26.0 ms
64 bytes from 2001:41d0:801:2000::539f: icmp_seq=2 ttl=46 time=27.2 ms
Upon discussing with a net-snmp github dev, he found out that a change to a feature introduced a char addr[64] to store the host address. They changed it to char addr* to work with >64 characters domains, and its working properly now.
This bug breaks some of our monitoring when updating our librenms ubuntu host to 22.04, which is an issue.
I am of course available if any additional information is needed.
Cheers,
Tanguy |
Hello all,
We recently encountered a bug in net-snmp V5.9.1 related to DNS name length (https://github.com/net-snmp/net-snmp/issues/476)
Here's a summary :
What I did :
snmpstatus -v2c -c public aaaaaaaaaaaaaaaaaaaaaaaaa.aaaaaaaaaaaaaaaaaaaaaaaaa.taledo.games
What I expected to happen :
Get a timeout from this domain because there's no snmpd running, and I wouldn't use public as a community anyway.
What I got :
snmpstatus: Unknown host (aaaaaaaaaaaaaaaaaaaaaaaaa.aaaaaaaaaaaaaaaaaaaaaaaaa.taledo.games) (No such file or directory)
This domain does exist, and does ping properly (also has V4, this is not the problem here):
PING aaaaaaaaaaaaaaaaaaaaaaaaa.aaaaaaaaaaaaaaaaaaaaaaaaa.taledo.games(2001:41d0:801:2000::539f) 56 data bytes
64 bytes from 2001:41d0:801:2000::539f: icmp_seq=1 ttl=46 time=26.0 ms
64 bytes from 2001:41d0:801:2000::539f: icmp_seq=2 ttl=46 time=27.2 ms
Upon discussing with a net-snmp github dev, they found out that a change to a feature introduced a char addr[64] to store the host address. They changed it to char addr* to work with >64 characters domains, and its working properly now.
This bug breaks some of our monitoring when updating our librenms ubuntu host to 22.04, which is an issue.
I am of course available if any additional information is needed.
Cheers,
Tanguy |
|
| 2022-12-02 18:00:35 |
Lena Voytek |
bug |
|
|
added subscriber Lena Voytek |
| 2022-12-02 18:04:41 |
Lena Voytek |
net-snmp (Ubuntu): status |
New |
Confirmed |
|
| 2022-12-02 18:04:49 |
Lena Voytek |
nominated for series |
|
Ubuntu Jammy |
|
| 2022-12-02 18:04:49 |
Lena Voytek |
bug task added |
|
net-snmp (Ubuntu Jammy) |
|
| 2022-12-02 18:04:49 |
Lena Voytek |
nominated for series |
|
Ubuntu Lunar |
|
| 2022-12-02 18:04:49 |
Lena Voytek |
bug task added |
|
net-snmp (Ubuntu Lunar) |
|
| 2022-12-02 18:04:49 |
Lena Voytek |
nominated for series |
|
Ubuntu Kinetic |
|
| 2022-12-02 18:04:49 |
Lena Voytek |
bug task added |
|
net-snmp (Ubuntu Kinetic) |
|
| 2022-12-02 18:04:58 |
Lena Voytek |
net-snmp (Ubuntu Jammy): status |
New |
Confirmed |
|
| 2022-12-02 18:05:07 |
Lena Voytek |
net-snmp (Ubuntu Lunar): status |
Confirmed |
New |
|
| 2022-12-05 15:37:50 |
Lena Voytek |
net-snmp (Ubuntu Lunar): status |
New |
In Progress |
|
| 2022-12-05 15:46:49 |
Launchpad Janitor |
merge proposal linked |
|
https://code.launchpad.net/~lvoytek/ubuntu/+source/net-snmp/+git/net-snmp/+merge/434101 |
|
| 2022-12-09 21:46:00 |
Lena Voytek |
description |
Hello all,
We recently encountered a bug in net-snmp V5.9.1 related to DNS name length (https://github.com/net-snmp/net-snmp/issues/476)
Here's a summary :
What I did :
snmpstatus -v2c -c public aaaaaaaaaaaaaaaaaaaaaaaaa.aaaaaaaaaaaaaaaaaaaaaaaaa.taledo.games
What I expected to happen :
Get a timeout from this domain because there's no snmpd running, and I wouldn't use public as a community anyway.
What I got :
snmpstatus: Unknown host (aaaaaaaaaaaaaaaaaaaaaaaaa.aaaaaaaaaaaaaaaaaaaaaaaaa.taledo.games) (No such file or directory)
This domain does exist, and does ping properly (also has V4, this is not the problem here):
PING aaaaaaaaaaaaaaaaaaaaaaaaa.aaaaaaaaaaaaaaaaaaaaaaaaa.taledo.games(2001:41d0:801:2000::539f) 56 data bytes
64 bytes from 2001:41d0:801:2000::539f: icmp_seq=1 ttl=46 time=26.0 ms
64 bytes from 2001:41d0:801:2000::539f: icmp_seq=2 ttl=46 time=27.2 ms
Upon discussing with a net-snmp github dev, they found out that a change to a feature introduced a char addr[64] to store the host address. They changed it to char addr* to work with >64 characters domains, and its working properly now.
This bug breaks some of our monitoring when updating our librenms ubuntu host to 22.04, which is an issue.
I am of course available if any additional information is needed.
Cheers,
Tanguy |
[Impact]
Currently, users who would like to query domains with a name length of 63 characters or more are unable to, and instead receive an Unknown Host error.
This fix should be backported to stable releases as net-snmp should be capable of querying all relevant domains, and this includes very long ones.
The bug is fixed by making the limited-size static string into a dynamic one.
[Test Plan]
The fix can be tested with the following:
# lxc launch images:ubuntu/jammy test-net-snmp
# lxc exec test-net-snmp bash
# apt update && apt dist-upgrade -y
# apt install snmp -y
- Attempt to connect to an existing domain containing over 63 characters:
# snmpstatus -v2c -c public aaaaaaaaaaaaaaaaaaaaaaaaa.aaaaaaaaaaaaaaaaaaaaaaaaa.taledo.games
- Before the fix, it will show the following
Created directory: /var/lib/snmp/cert_indexes
snmpstatus: Unknown host (aaaaaaaaaaaaaaaaaaaaaaaaa.aaaaaaaaaaaaaaaaaaaaaaaaa.taledo.games) (Resource temporarily unavailable)
- After the fix, it succeeds or correctly shows that snmpd is not running on the domain
Timeout: No Response from aaaaaaaaaaaaaaaaaaaaaaaaa.aaaaaaaaaaaaaaaaaaaaaaaaa.taledo.games
[Where problems could occur]
Since this change swaps from static allocation to dynamic to handle larger addresses, regressions would most likely be related to memory leaks. If this change fails to account for all situations, the address variable may not be freed, leading to greater memory usage. Also, without the size limit, overwhelmingly large address inputs may also cause issues.
[Original Description]
Hello all,
We recently encountered a bug in net-snmp V5.9.1 related to DNS name length (https://github.com/net-snmp/net-snmp/issues/476)
Here's a summary :
What I did :
snmpstatus -v2c -c public aaaaaaaaaaaaaaaaaaaaaaaaa.aaaaaaaaaaaaaaaaaaaaaaaaa.taledo.games
What I expected to happen :
Get a timeout from this domain because there's no snmpd running, and I wouldn't use public as a community anyway.
What I got :
snmpstatus: Unknown host (aaaaaaaaaaaaaaaaaaaaaaaaa.aaaaaaaaaaaaaaaaaaaaaaaaa.taledo.games) (No such file or directory)
This domain does exist, and does ping properly (also has V4, this is not the problem here):
PING aaaaaaaaaaaaaaaaaaaaaaaaa.aaaaaaaaaaaaaaaaaaaaaaaaa.taledo.games(2001:41d0:801:2000::539f) 56 data bytes
64 bytes from 2001:41d0:801:2000::539f: icmp_seq=1 ttl=46 time=26.0 ms
64 bytes from 2001:41d0:801:2000::539f: icmp_seq=2 ttl=46 time=27.2 ms
Upon discussing with a net-snmp github dev, they found out that a change to a feature introduced a char addr[64] to store the host address. They changed it to char addr* to work with >64 characters domains, and its working properly now.
This bug breaks some of our monitoring when updating our librenms ubuntu host to 22.04, which is an issue.
I am of course available if any additional information is needed.
Cheers,
Tanguy |
|
| 2022-12-09 21:46:05 |
Lena Voytek |
net-snmp (Ubuntu Kinetic): status |
New |
In Progress |
|
| 2022-12-09 21:46:09 |
Lena Voytek |
net-snmp (Ubuntu Jammy): status |
Confirmed |
In Progress |
|
| 2022-12-09 21:53:53 |
Launchpad Janitor |
merge proposal linked |
|
https://code.launchpad.net/~lvoytek/ubuntu/+source/net-snmp/+git/net-snmp/+merge/434455 |
|
| 2022-12-09 21:54:37 |
Launchpad Janitor |
merge proposal linked |
|
https://code.launchpad.net/~lvoytek/ubuntu/+source/net-snmp/+git/net-snmp/+merge/434456 |
|
| 2022-12-13 04:12:10 |
Launchpad Janitor |
net-snmp (Ubuntu Lunar): status |
In Progress |
Fix Released |
|
| 2022-12-13 14:20:31 |
Lena Voytek |
merge proposal unlinked |
https://code.launchpad.net/~lvoytek/ubuntu/+source/net-snmp/+git/net-snmp/+merge/434455 |
|
|
| 2022-12-13 14:25:17 |
Launchpad Janitor |
merge proposal linked |
|
https://code.launchpad.net/~lvoytek/ubuntu/+source/net-snmp/+git/net-snmp/+merge/434568 |
|
| 2022-12-13 14:26:10 |
Lena Voytek |
net-snmp (Ubuntu Jammy): assignee |
|
Lena Voytek (lvoytek) |
|
| 2022-12-13 14:26:13 |
Lena Voytek |
net-snmp (Ubuntu Kinetic): assignee |
|
Lena Voytek (lvoytek) |
|
| 2022-12-13 14:26:15 |
Lena Voytek |
net-snmp (Ubuntu Lunar): assignee |
|
Lena Voytek (lvoytek) |
|
| 2022-12-16 07:55:54 |
Timo Aaltonen |
net-snmp (Ubuntu Kinetic): status |
In Progress |
Fix Committed |
|
| 2022-12-16 07:55:55 |
Timo Aaltonen |
bug |
|
|
added subscriber Ubuntu Stable Release Updates Team |
| 2022-12-16 07:55:57 |
Timo Aaltonen |
bug |
|
|
added subscriber SRU Verification |
| 2022-12-16 07:55:59 |
Timo Aaltonen |
tags |
|
verification-needed verification-needed-kinetic |
|
| 2022-12-16 07:56:41 |
Timo Aaltonen |
net-snmp (Ubuntu Jammy): status |
In Progress |
Fix Committed |
|
| 2022-12-16 07:56:45 |
Timo Aaltonen |
tags |
verification-needed verification-needed-kinetic |
verification-needed verification-needed-jammy verification-needed-kinetic |
|
| 2022-12-16 11:43:47 |
Tanguy Pelado |
tags |
verification-needed verification-needed-jammy verification-needed-kinetic |
verification-done-jammy verification-needed verification-needed-kinetic |
|
| 2023-01-04 14:46:34 |
Lena Voytek |
tags |
verification-done-jammy verification-needed verification-needed-kinetic |
verification-done verification-done-jammy verification-done-kinetic |
|
| 2023-01-05 16:24:48 |
Robie Basak |
tags |
verification-done verification-done-jammy verification-done-kinetic |
block-proposed-jammy verification-done verification-done-jammy verification-done-kinetic |
|
| 2023-01-05 16:24:55 |
Robie Basak |
tags |
block-proposed-jammy verification-done verification-done-jammy verification-done-kinetic |
block-proposed-jammy block-proposed-kinetic verification-done verification-done-jammy verification-done-kinetic |
|
| 2023-01-05 21:43:25 |
Launchpad Janitor |
merge proposal linked |
|
https://code.launchpad.net/~lvoytek/ubuntu/+source/net-snmp/+git/net-snmp/+merge/435225 |
|
| 2023-01-11 16:58:25 |
Launchpad Janitor |
merge proposal linked |
|
https://code.launchpad.net/~sergiodj/ubuntu/+source/net-snmp/+git/net-snmp/+merge/435597 |
|
| 2023-01-11 16:59:24 |
Sergio Durigan Junior |
merge proposal unlinked |
https://code.launchpad.net/~sergiodj/ubuntu/+source/net-snmp/+git/net-snmp/+merge/435597 |
|
|
| 2023-01-23 18:26:20 |
Launchpad Janitor |
merge proposal linked |
|
https://code.launchpad.net/~lvoytek/ubuntu/+source/net-snmp/+git/net-snmp/+merge/436203 |
|
| 2023-01-23 18:35:46 |
Launchpad Janitor |
merge proposal linked |
|
https://code.launchpad.net/~lvoytek/ubuntu/+source/net-snmp/+git/net-snmp/+merge/436205 |
|
| 2023-01-23 18:38:54 |
Lena Voytek |
net-snmp (Ubuntu Jammy): status |
Fix Committed |
In Progress |
|
| 2023-01-23 18:38:56 |
Lena Voytek |
net-snmp (Ubuntu Kinetic): status |
Fix Committed |
In Progress |
|
| 2023-01-25 17:08:33 |
Lena Voytek |
tags |
block-proposed-jammy block-proposed-kinetic verification-done verification-done-jammy verification-done-kinetic |
|
|
| 2023-02-16 14:05:23 |
Andreas Hasenack |
net-snmp (Ubuntu Kinetic): status |
In Progress |
Fix Committed |
|
| 2023-02-16 14:05:30 |
Andreas Hasenack |
tags |
|
verification-needed verification-needed-kinetic |
|
| 2023-02-16 14:06:41 |
Andreas Hasenack |
net-snmp (Ubuntu Jammy): status |
In Progress |
Fix Committed |
|
| 2023-02-16 14:06:47 |
Andreas Hasenack |
tags |
verification-needed verification-needed-kinetic |
verification-needed verification-needed-jammy verification-needed-kinetic |
|
| 2023-02-16 17:38:30 |
Lena Voytek |
tags |
verification-needed verification-needed-jammy verification-needed-kinetic |
verification-done-jammy verification-needed verification-needed-kinetic |
|
| 2023-02-16 17:40:50 |
Lena Voytek |
tags |
verification-done-jammy verification-needed verification-needed-kinetic |
verification-done verification-done-jammy verification-done-kinetic |
|
| 2023-03-01 06:44:15 |
Chris Halse Rogers |
removed subscriber Ubuntu Stable Release Updates Team |
|
|
|
| 2023-03-01 06:49:07 |
Launchpad Janitor |
net-snmp (Ubuntu Kinetic): status |
Fix Committed |
Fix Released |
|
| 2023-03-01 06:49:43 |
Launchpad Janitor |
net-snmp (Ubuntu Jammy): status |
Fix Committed |
Fix Released |
|
| 2024-01-02 21:29:49 |
Launchpad Janitor |
merge proposal linked |
|
https://code.launchpad.net/~sergiodj/ubuntu/+source/net-snmp/+git/net-snmp/+merge/457859 |
|
| 2024-01-02 21:30:36 |
Sergio Durigan Junior |
merge proposal unlinked |
https://code.launchpad.net/~sergiodj/ubuntu/+source/net-snmp/+git/net-snmp/+merge/457859 |
|
|
