Page MenuHomeFreeBSD

flua: Fix SIGSEGV in lua_chown when uid/gid doesn't exist
ClosedPublic

Authored by dtxdf on Feb 8 2026, 12:01 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Mar 22, 9:05 PM
Unknown Object (File)
Fri, Mar 13, 6:54 AM
Unknown Object (File)
Thu, Mar 12, 4:58 PM
Unknown Object (File)
Sat, Feb 28, 4:38 PM
Unknown Object (File)
Feb 24 2026, 9:47 AM
Unknown Object (File)
Feb 22 2026, 9:26 PM
Unknown Object (File)
Feb 16 2026, 9:20 PM
Unknown Object (File)
Feb 16 2026, 8:43 AM
Subscribers

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

dtxdf requested review of this revision.Feb 8 2026, 12:01 AM

Were you planning on pushing this into the src repo?
I'd add a note that says that both getpwnam_r and getgrnam_r return 0/NULL in this case to the commit message. If you add that, you can add 'Approved by: imp' to the commit since I don't think you have a src commit bit.
And we prefer you use -U99999 to get the context. Not a huge deal for this one, but something to note in the future.
If you'd prefer someone else commit this, I can do the deed.

This revision is now accepted and ready to land.Feb 8 2026, 5:55 PM

Committed!

In D55172#1261760, @imp wrote:

And we prefer you use -U99999 to get the context. Not a huge deal for this one, but something to note in the future

And thanks for the tip about -U99999, I'll use it in the next case!

Committed!

Thanks for doing that!

In D55172#1261760, @imp wrote:

And we prefer you use -U99999 to get the context. Not a huge deal for this one, but something to note in the future

And thanks for the tip about -U99999, I'll use it in the next case!

You bet. Though if you do more than a couple more of these, you'd be better off getting 'git arc' working. Having branches, and rebasing and git arc update main..HEAD makes more complex submissions a breeze.