Page MenuHomeFreeBSD

heimdal: Add 64-bit integer support to ASN.1 compiler, part 1
ClosedPublic

Authored by dim on Apr 16 2024, 7:13 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Nov 14, 3:14 PM
Unknown Object (File)
Wed, Nov 6, 12:33 AM
Unknown Object (File)
Wed, Nov 6, 12:33 AM
Unknown Object (File)
Wed, Nov 6, 12:20 AM
Unknown Object (File)
Oct 21 2024, 5:25 AM
Unknown Object (File)
Sep 6 2024, 12:43 AM
Unknown Object (File)
Aug 10 2024, 4:12 PM
Unknown Object (File)
Aug 6 2024, 10:54 AM
Subscribers

Details

Reviewers
emaste
cy
Summary

Import upstream 19d378f44:

ASN.1 INTEGERs will now compile to C int64_t or uint64_t, depending
on whether the constraint ranges include numbers that cannot be
represented in 32-bit ints and whether they include negative
numbers.

Template backend support included.  check-template is now built with
--template, so we know we're testing it.

Tests included.

This is a direct application of the upstream patch, but it DOES NOT compile.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

dim requested review of this revision.Apr 16 2024, 7:13 PM

There is no need to touch Makefile.am. We don't use it.

crypto/heimdal/lib/asn1/test.asn1
11

We don't use test.asn1.

crypto/heimdal/lib/asn1/test.asn1
11

IMO it makes sense to just apply the upstream patch though, there's not much value in going out of our way to undo part of it.

Except for the files we don't use, LGTM.

This revision is now accepted and ready to land.Apr 17 2024, 3:21 AM