Page MenuHomeFreeBSD
Differential D16641

Make ldconfig(8) atomic
ClosedPublic
Actions

Authored by trasz on Aug 9 2018, 11:13 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Feb 11, 10:32 AM
Unknown Object (File)
Sat, Feb 8, 4:05 PM
Unknown Object (File)
Fri, Feb 7, 11:39 PM
Unknown Object (File)
Jan 12 2025, 4:34 PM
Unknown Object (File)
Jan 9 2025, 6:10 PM
Unknown Object (File)
Jan 5 2025, 3:36 PM
Unknown Object (File)
Dec 4 2024, 8:38 AM
Unknown Object (File)
Nov 29 2024, 5:16 PM
View All 72 Files
Subscribers
imp

Details

Reviewers
imp
kib
Commits
rS337530: Make ldconfig(8) atomic, by removing an unneccessary call to unlink(2)
Summary

Make ldconfig(8) atomic, by removing an unneccessary call to unlink(2)
before rename(2).

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

trasz created this revision.Aug 9 2018, 11:13 AM
Herald added a subscriber: imp. · View Herald TranscriptAug 9 2018, 11:13 AM
Harbormaster completed remote builds in B18662: Diff 46455.Aug 9 2018, 11:13 AM
trasz added reviewers: imp, kib.Aug 9 2018, 11:14 AM
trasz updated this revision to Diff 46456.Aug 9 2018, 11:15 AM

Spurious empty line.

Harbormaster completed remote builds in B18663: Diff 46456.Aug 9 2018, 11:15 AM
kib accepted this revision.Aug 9 2018, 11:24 AM
This revision is now accepted and ready to land.Aug 9 2018, 11:24 AM
kib added inline comments.Aug 9 2018, 11:26 AM
sbin/ldconfig/ldconfig.c
522 ↗(On Diff #46456)

This comment really marks both unlink() and rename() calls and could be preserved. It is arguably not too useful.

Closed by commit rS337530: Make ldconfig(8) atomic, by removing an unneccessary call to unlink(2) (authored by trasz). · Explain WhyAug 9 2018, 11:46 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
sbin/
ldconfig/
7 lines

Diff 46458

View Options

head/sbin/ldconfig/ldconfig.c

Loading...