fieldClassifier: bugfix: asn overflow after saved to uint16

Tomas-Vrana · Tomas-Vrana · commit 0e8ae1240bcf · 2025-11-25T17:00:51.000+01:00
changelog: Bugfixes in &lt;fieldClassifier&gt;.
diff --git a/modules/fieldClassifier/src/plugins/asn.cpp b/modules/fieldClassifier/src/plugins/asn.cpp
@@ -30,7 +30,7 @@ void ASN::storeParameters(argparse::ArgumentParser& parser)
 FieldDefinition ASN::defineFields()
 {
 	// list of all fields provided by this plugin
-	FieldDefinition allFields = {{"ASN", DataType::UINT16}, {"ASO", DataType::STRING}};
+	FieldDefinition allFields = {{"ASN", DataType::UINT32}, {"ASO", DataType::STRING}};
 
 	return allFields;
 }
@@ -53,7 +53,7 @@ bool ASN::getData(DataMap& dataMap, std::string& ipAddr)
 	}
 	if (dataMap.find("ASN") != dataMap.end()) {
 		err = MMDB_get_value(&result.entry, &entryData, "autonomous_system_number", NULL);
-		dataMap.at("ASN") = checkEntryData() ? entryData.uint16 : EMPTY_UINT16;
+		dataMap.at("ASN") = checkEntryData() ? entryData.uint32 : EMPTY_UINT16;
 	}
 	if (dataMap.find("ASO") != dataMap.end()) {
 		err = MMDB_get_value(&result.entry, &entryData, "autonomous_system_organization", NULL);

Original file line number	Diff line number	Diff line change
`@@ -30,7 +30,7 @@ void ASN::storeParameters(argparse::ArgumentParser& parser)`
`30`	`30`	`FieldDefinition ASN::defineFields()`
`31`	`31`	`{`
`32`	`32`	`// list of all fields provided by this plugin`
`33`		`- FieldDefinition allFields = {{"ASN", DataType::UINT16}, {"ASO", DataType::STRING}};`
	`33`	`+ FieldDefinition allFields = {{"ASN", DataType::UINT32}, {"ASO", DataType::STRING}};`
`34`	`34`
`35`	`35`	`return allFields;`
`36`	`36`	`}`
`@@ -53,7 +53,7 @@ bool ASN::getData(DataMap& dataMap, std::string& ipAddr)`
`53`	`53`	`}`
`54`	`54`	`if (dataMap.find("ASN") != dataMap.end()) {`
`55`	`55`	`err = MMDB_get_value(&result.entry, &entryData, "autonomous_system_number", NULL);`
`56`		`- dataMap.at("ASN") = checkEntryData() ? entryData.uint16 : EMPTY_UINT16;`
	`56`	`+ dataMap.at("ASN") = checkEntryData() ? entryData.uint32 : EMPTY_UINT16;`
`57`	`57`	`}`
`58`	`58`	`if (dataMap.find("ASO") != dataMap.end()) {`
`59`	`59`	`err = MMDB_get_value(&result.entry, &entryData, "autonomous_system_organization", NULL);`