Page MenuHomeFreeBSD
Differential D34550

bsnmp: Use mkstemp when creating clients local socket
ClosedPublic
Actions

Authored by thj on Mar 14 2022, 3:39 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Sep 8, 5:09 PM
Unknown Object (File)
Sat, Sep 5, 8:35 PM
Unknown Object (File)
Sat, Sep 5, 11:04 AM
Unknown Object (File)
Sat, Sep 5, 4:14 AM
Unknown Object (File)
Fri, Sep 4, 8:50 PM
Unknown Object (File)
Thu, Sep 3, 10:45 PM
Unknown Object (File)
Thu, Sep 3, 1:10 PM
Unknown Object (File)
Wed, Sep 2, 9:04 PM
View All Files
Subscribers
imp

Details

Reviewers
harti
rew
Group Reviewers
Klara
Commits
rG81e0e7b9e36d: bsnmp: Use mkstemp when creating clients local socket

Diff Detail

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

Event Timeline

thj created this revision.Mar 14 2022, 3:39 PM
Herald added a subscriber: imp. · View Herald TranscriptMar 14 2022, 3:39 PM
thj requested review of this revision.Mar 14 2022, 3:39 PM
Harbormaster completed remote builds in B44764: Diff 103828.Mar 14 2022, 3:39 PM
thj added a comment.Mar 14 2022, 3:41 PM

I am not sure if FreeBSD is upstream here, it might be harti's own private tree, it isn't 100% clear to me.

I have created this review as a place for discussion if we need it.

harti accepted this revision.Mar 14 2022, 6:03 PM

Looks good to me.

Yes, upstream is currently a private repo which I plan to make public later this year when I have v2 in a working condition. I normally fetch all modifications from the FreeBSD tree and put them into that repo.

This revision is now accepted and ready to land.Mar 14 2022, 6:03 PM
rew accepted this revision.Mar 14 2022, 6:32 PM
Closed by commit rG81e0e7b9e36d: bsnmp: Use mkstemp when creating clients local socket (authored by thj). · Explain WhyMar 15 2022, 8:21 AM
This revision was automatically updated to reflect the committed changes.
thj added a commit: rG81e0e7b9e36d: bsnmp: Use mkstemp when creating clients local socket.
glebius added a reverting change: D51031: libbsnmp: fix local client socket creation.Jun 25 2025, 2:17 AM
glebius mentioned this in D51031: libbsnmp: fix local client socket creation.Jun 25 2025, 2:18 AM
glebius added a reverting change: rGd4f042f094ec: libbsnmp: fix local client socket creation.Jul 17 2025, 3:59 PM

Revision Contents
Changeset List

PathSize
contrib/
bsnmp/
lib/
2 lines

Diff 103866

View Options

contrib/bsnmp/lib/snmpclient.c

Loading...