Bug 1035 – Inclusion of stdint.h instead of inttypes.h breaks build on Solaris 9

Bug 1035 - Inclusion of stdint.h instead of inttypes.h breaks build on Solaris 9


Summary:	Inclusion of stdint.h instead of inttypes.h breaks build on Solaris 9

Status:	RESOLVED FIXED

Product:	Sudo
Classification:	Unclassified
Component:	Sudo
Version:	1.9.11
Hardware:	Sun Solaris 2.x

Importance:	low low
Assigned To:	Todd C. Miller

URL:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2022-06-27 06:21 MDT by Dagobert Michelsen
Modified:	2022-10-23 09:13 MDT (History)
CC List:	0 users

See Also:

Attachments
Use inttypes.h instead of stdint.h (647 bytes, application/octet-stream) 2022-06-27 06:21 MDT, Dagobert Michelsen	Details
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Dagobert Michelsen 2022-06-27 06:21:49 MDT

Created attachment 567 [details]
Use inttypes.h instead of stdint.h

Hi,

the current sudo 1.9.11p3 uses stdint.h which does exist on Solaris 9. Replacing the include with inttypes.h resolves the issue as can be seen from the attached patch.

The CI for Solaris did not catch the error, but the build dates also don't seem up to date. Is there a different repo I should attach to?
  https://buildfarm.opencsw.org/buildbot/waterfall?category=sudo


Best regards

  -- Dago

Comment 1 Todd C. Miller 2022-06-27 07:27:06 MDT

Thanks, I've fixed this in https://www.sudo.ws/repos/sudo/rev/da6185c4c418

https://www.sudo.ws/repos/sudo is the correct repo but you can also use github if that is easier, https://github.com/sudo-project/sudo/

Comment 2 Todd C. Miller 2022-10-23 09:13:38 MDT

Fixed in sudo 1.9.12, available now.