Page MenuHomeFreeBSD

Make pseudofs(9) create directory entries in order
ClosedPublic

Authored by trasz on Sep 6 2019, 1:07 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Nov 23, 12:19 PM
Unknown Object (File)
Sat, Nov 23, 8:38 AM
Unknown Object (File)
Oct 25 2024, 2:26 PM
Unknown Object (File)
Oct 2 2024, 5:01 AM
Unknown Object (File)
Sep 24 2024, 4:13 PM
Unknown Object (File)
Sep 23 2024, 6:50 PM
Unknown Object (File)
Sep 20 2024, 11:04 PM
Unknown Object (File)
Sep 20 2024, 11:04 PM
Subscribers

Details

Summary

Make pseudofs(9) create directory entries in order, instead
of the reverse.

This fixes Linux sysctl(8) binary - it assumes the first two
directory entries are always "." and "..". There might be other
Linux apps affected by this.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 26309
Build 24788: arc lint + arc unit

Event Timeline

sys/fs/pseudofs/pseudofs.c
145

I think you do not need this check, instead fold it with the previous pn_nodes == NULL branch. KASSERT() there demonstrates it nicely.

172

What if pn_next never equal to pn ?

trasz marked an inline comment as done.

Fixes from kib.

Thanks; does the updated version look ok?

Did you considered switching to sys/queue.h ?

This revision is now accepted and ready to land.Sep 9 2019, 12:41 PM

I've thought about it; I guess des@ hadn't used that to avoid contaminating pseudofs headers with queue.h, and I didn't want to change it.

I've thought about it; I guess des@ hadn't used that to avoid contaminating pseudofs headers with queue.h, and I didn't want to change it.

Lets take a look. Very first line of pseudofs.h includes sys/jail.h. In sys/jail.h line 121 we see inclusion of sys/queue.h.