Page MenuHomeFreeBSD

Replacing iterating over rootpath by strsep
ClosedPublic

Authored by kczekirda on May 14 2017, 5:07 PM.
Tags
None
Referenced Files
F87213643: D10726.id28585.diff
Sun, Jun 30, 8:28 AM
Unknown Object (File)
Tue, Jun 25, 6:26 AM
Unknown Object (File)
May 20 2024, 6:29 AM
Unknown Object (File)
Apr 29 2024, 9:10 AM
Unknown Object (File)
Apr 5 2024, 3:31 PM
Unknown Object (File)
Feb 20 2024, 6:44 PM
Unknown Object (File)
Feb 2 2024, 12:10 AM
Unknown Object (File)
Jan 19 2024, 6:16 AM
Subscribers

Details

Summary

Replacing iteration over chars array by strsep.

Sponsored by: Oktawave

Test Plan

Tested on:

option root-path "192.168.22.1:/tftproot";
option root-path "/tftproot";

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

kczekirda edited the summary of this revision. (Show Details)

I reviewed that earlier.
If you guys accept that I will commit it.

This revision is now accepted and ready to land.May 14 2017, 5:51 PM
jhb added inline comments.
sys/boot/common/dev_net.c
397

Not sure if strlcpy() is defined to handle overlapping regions? In that case I think you may want to stick with memmove/bcopy here as those handle overlapping copies.

This revision now requires review to proceed.May 19 2017, 9:22 PM
jhb added inline comments.
sys/boot/common/dev_net.c
397

Thanks, my last suggestion / nit would be to fix the old style bug since you are changing this line anyway and put spaces around the '+'.

This revision is now accepted and ready to land.May 22 2017, 8:05 PM
This revision was automatically updated to reflect the committed changes.