Page MenuHomeFreeBSD
Differential D20141

Modify ifname_linux_to_bsd() to properly work with Linux loopback interface
ClosedPublic
Actions

Authored by dchagin on May 2 2019, 2:29 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Dec 19, 4:02 PM
Unknown Object (File)
Oct 8 2024, 1:40 PM
Unknown Object (File)
Oct 8 2024, 1:40 PM
Unknown Object (File)
Oct 8 2024, 1:40 PM
Unknown Object (File)
Oct 8 2024, 1:40 PM
Unknown Object (File)
Oct 8 2024, 1:09 PM
Unknown Object (File)
Sep 26 2024, 7:47 PM
Unknown Object (File)
Sep 23 2024, 4:16 PM
View All 49 Files
Subscribers
emaste
imp

Details

Reviewers
emaste
Group Reviewers
Linux Emulation
Commits
rS347202: Complete r347052 (https://reviews.freebsd.org/D20137) as it it was not
Summary

Modify ifname_linux_to_bsd() to properly work with Linux loopback
interface (for future use in linsysfs).

Allow NULL bsdname to aviod excess strlcpy if bsdname is not needed
by the caller.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

dchagin created this revision.May 2 2019, 2:29 PM
Herald added a subscriber: imp. · View Herald TranscriptMay 2 2019, 2:29 PM
Harbormaster completed remote builds in B24066: Diff 56961.May 2 2019, 2:29 PM
dchagin added a reviewer: Linux Emulation.May 2 2019, 2:30 PM
emaste added a subscriber: emaste.May 3 2019, 9:13 PM
emaste added inline comments.
sys/compat/linux/linux.c
239

especially now that is_lo is bool I think the ?: operator is unnecessary obfuscation

dchagin updated this revision to Diff 57020.May 3 2019, 9:28 PM
dchagin retitled this revision from odify ifname_linux_to_bsd() to properly work with Linux loopback interface to Modify ifname_linux_to_bsd() to properly work with Linux loopback interface.
dchagin marked an inline comment as done.
dchagin added inline comments.
sys/compat/linux/linux.c
239

Yes, indeed, thank you:)

emaste accepted this revision.May 6 2019, 6:26 PM

LGTM

sys/compat/linux/linux.c
234

lxname[len] == '\0' would be more correct?

This revision is now accepted and ready to land.May 6 2019, 6:26 PM
Closed by commit rS347202: Complete r347052 (https://reviews.freebsd.org/D20137) as it it was not (authored by dchagin). · Explain WhyMay 6 2019, 7:56 PM
This revision was automatically updated to reflect the committed changes.
dchagin marked an inline comment as done.

Revision Contents
Changeset List

PathSize
sys/
compat/
linux/
20 lines

Diff 57020

View Options

sys/compat/linux/linux.c

Loading...