Summary

current-user-id setter from (chicken process-context posix) maybe needs some argument validation

Metadata

• Id: e1ce4f5171a484d0a21d03dce01c701447a4a585
• Trac id: 1787
• Type: defect
• Reporter: teiresias
• Owner:
• Cc:
• Status: closed
• Component: core libraries
• Estimated difficulty:
• Resolution: fixed
• Priority: minor
• Milestone: 5.3
• Version: 5.2.0
• Changetime: 2021-09-25 06:11:30 UTC
• Created: 2021-09-24 06:59:37 UTC
• Keywords:

Description

 #;2> (set! (current-user-id) "abracadabra")
 #;3> (current-user-id)
 2245824928

This should fail with an error rather than setting the uid to a bogus value. Maybe the current-user-id setter could be made to take a string and do a uid lookup, but that's a discussion for another day.

Changes and comments

[2021-09-24 08:18:36 UTC] sjamaan changed milestone from someday to 5.3

[2021-09-24 08:18:36 UTC] sjamaan wrote:

Trivial enough to fix for 5.3.0

[2021-09-25 06:11:30 UTC] evhan changed status from new to closed

[2021-09-25 06:11:30 UTC] evhan set resolution to fixed

[2021-09-25 06:11:30 UTC] evhan wrote:

Fixed by 95ea81bd.