RHEL Engineering is moving the tracking of its product development work on RHEL 6 through RHEL 9 to Red Hat Jira (issues.redhat.com). If you're a Red Hat customer, please continue to file support cases via the Red Hat customer portal. If you're not, please head to the "RHEL project" in Red Hat Jira and file new tickets here. Individual Bugzilla bugs in the statuses "NEW", "ASSIGNED", and "POST" are being migrated throughout September 2023. Bugs of Red Hat partners with an assigned Engineering Partner Manager (EPM) are migrated in late September as per pre-agreed dates. Bugs against components "kernel", "kernel-rt", and "kpatch" are only migrated if still in "NEW" or "ASSIGNED". If you cannot log in to RH Jira, please consult article #7032570. That failing, please send an e-mail to the RH Jira admins at rh-issues@redhat.com to troubleshoot your issue as a user management inquiry. The email creates a ServiceNow ticket with Red Hat. Individual Bugzilla bugs that are migrated will be moved to status "CLOSED", resolution "MIGRATED", and set with "MigratedToJIRA" in "Keywords". The link to the successor Jira issue will be found under "Links", have a little "two-footprint" icon next to it, and direct you to the "RHEL project" in Red Hat Jira (issue links are of type "https://issues.redhat.com/browse/RHEL-XXXX", where "X" is a digit). This same link will be available in a blue banner at the top of the page informing you that that bug has been migrated.

Description of problem:

snmp_set_var_typed_value function for ASN_IPADDRESS sets wrong size

Version-Release number of selected component (if applicable):
net-snmp-5.5-44.el6_4.2

How reproducible:
100%

Steps to Reproduce:
1. Call snmp_set_var_typed_value() with ip address in args for ASN_IPADDRESS to build a trap
2. send the trap
3. sink cannot parse the packet

Actual results:
4

Expected results:
sizeof(in_addr_t)

Additional info:
Previous version was OK (net-snmp-5.5-44.el6_4.1), net-snmp-5.5-ber-int-size.patch brokes the lib:

---
-        case ASN_OCTET_STR:
         case ASN_IPADDRESS:
+            if (vp->val_len != 4)
+                return -1;
+            /* fallthrough */
+        case ASN_OCTET_STR:
         case ASN_OPAQUE:
         case ASN_NSAP:
             if (vp->val_len < sizeof(vp->buf)) {
---

should be

if (vp->val_len != sizeof(in_addr_t))

(In reply to Konstantin Volkov from comment #0)
> should be
> 
> if (vp->val_len != sizeof(in_addr_t))

Pardon my ignorance, but on which architecture is sizeof(in_addr_t) != 4 ?

Of cause, sizeof(in_addr_t) always = 4.

In our code we passed next:

---
u_long n = htonl(src_);
snmp_set_var_typed_value(&dst_, ASN_IPADDRESS, (u_char* )&n, sizeof(n));
---

and it works OK with previous net_snmp version.

My fault, please close.

Thanks for the clarification, closing.