summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* More consistent loggingJakub Jankowski2018-05-241-2/+2
| | | | Signed-off-by: Jakub Jankowski <shasta@toxcorp.com>
* Quote variables in testsJakub Jankowski2018-05-241-3/+3
| | | | Signed-off-by: Jakub Jankowski <shasta@toxcorp.com>
* Use pkill/pgrepJakub Jankowski2018-05-241-4/+2
| | | | | | | Instead of heavy subshelling, use pkill/pgrep. Also, be a bit more specific on what we're killing. Signed-off-by: Jakub Jankowski <shasta@toxcorp.com>
* Replace -a / -o in conditionals with && / ||Jakub Jankowski2018-05-241-3/+3
| | | | | | | | | Conditional expression like [ test1 -a test2 ] should be avoided as it is not well defined. See http://pubs.opengroup.org/onlinepubs/9699919799/utilities/test.html#tag_20_128_16 and https://github.com/koalaman/shellcheck/wiki/SC2166 Signed-off-by: Jakub Jankowski <shasta@toxcorp.com>
* Take HWADDR from sysfsJakub Jankowski2018-05-241-1/+1
| | | | | | | Instead of "ip | sed", just read the sysfs file in which hardware address is readily presented for us. Signed-off-by: Jakub Jankowski <shasta@toxcorp.com>
* is_wireless_device: fix UUoC and reorderJakub Jankowski2018-05-241-3/+3
| | | | | | | | | | | | | There's no need to $(cat .../uevent | grep DEVTYPE) and then compare it to what we're looking for. grep can do all of it for us ("Useless Use of Cat"). While at it, if we have three ways of checking whether a device is wireless, let's order them from least expensive (checking dir is only one stat()) to most expensive (iwconfig | grep). Proof of correctness: if "iwconfig | grep" is returning true and the other two methods are not, we will still reach "iwconfig | grep" in the series of if/elif. Signed-off-by: Jakub Jankowski <shasta@toxcorp.com>
* netconfig: formatting tweaks plus fixup NM default usagePatrick J. Volkerding2018-05-241-24/+29
| | | | Signed-off-by: Robby Workman <rworkman@slackware.com>
* rc.inet1: set the bcast addr and flush ip addr on shutdowncurrent-20180318Robby Workman2018-03-181-2/+4
| | | | | | Thanks to Markus Wiesner. Signed-off-by: Robby Workman <rworkman@slackware.com>
* netconfig: Don't write hostname to NM config file any moreAndrzej Telszewski2018-03-181-4/+0
| | | | Signed-off-by: Robby Workman <rworkman@slackware.com>
* manpages/*: Sync with public -current treeRobby Workman2018-03-182-2/+4
|
* rc.inet1: Fix typo ( $1 --> $i )current-20171128Jakub Jankowski2017-11-261-1/+1
| | | | Signed-off-by: Robby Workman <rworkman@slackware.com>
* netconfig: Add back a dropped "."Robby Workman2017-11-261-1/+1
|
* rc.inet1: Check for iface in /sys/class/net, not /proc/net/devJakub Jankowski2017-11-211-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There is a slight problem with things like grep lo: /proc/net/dev especially with tun/tap devices with can have arbitrary names. Consider a scenario where you name your tap iface "trololo": $ grep -c lo: /proc/net/dev 2 $ To fix this (quite hypothetical, but still) problem, instead of simple grepping in /proc/net/dev, check of sysfs path existence (/sys/class/net/<interface>). Same fix in if_up() and if_down() for non-loopback interfaces. While there, also strip colon-whatever from interface name (what I think was the original idea behind a construct like grep $(echo ${1}: | cut -f 1 -d :): /proc/net/dev) by using parameter expansion: ${1%%:*} will remove the longest suffix matching ":*", ie. colon-and-everything-after. This is supported by POSIX: http://pubs.opengroup.org/onlinepubs/009695399/utilities/xcu_chap02.html Alternative solution would be to change the grep pattern to grep '^ *lo:' /proc/net/dev Signed-off-by: Robby Workman <rworkman@slackware.com>
* rc.inet1: Remove unnecessary unset numJakub Jankowski2017-11-211-2/+0
| | | | | | The only places using "num" already clear it with "num=0" Signed-off-by: Robby Workman <rworkman@slackware.com>
* netconfig: Add comment to netconfig-generated rc.inet1.confJakub Jankowski2017-11-211-0/+2
| | | | | | Add same comment as in 4561bd10747935edf3e4f17ad18768d17ee95a4e Signed-off-by: Robby Workman <rworkman@slackware.com>
* rc.inet1: Factor DEBUG_ETH_UP logging to a separate functionJakub Jankowski2017-11-211-15/+12
| | | | Signed-off-by: Robby Workman <rworkman@slackware.com>
* rc.inet1: Use ${array[*]} not ${array[@]} to produce a stringJakub Jankowski2017-11-211-1/+1
| | | | | | | | | | | | | | | This is the correct usage to produce one string from all elements of an array. foo[0]=b foo[1]=a foo[2]=r "${foo[@]}" here would mean "b" "a" "r" "${foo[*]}" however: "b a r", and that's what we mean when logging it. Signed-off-by: Robby Workman <rworkman@slackware.com>
* rc.inet1: Avoid de-/configuring ifaces with indexes >=MAXNICSJakub Jankowski2017-11-211-0/+12
| | | | | | | | | | | | | | | | | | | | | The code around MAXNICS currently only uses it to populate first $MAXNICS elements of IFNAME array, which is later used to match interface name to index used to access all other arrays. If you configure IPADDR[6], this code doesn't find a matching entry in IFNAME but doesn't do anything about it. This works by accident, because iteration variable "i" stays at "6" after exiting the loop. But when you configure IPADDR[7], it will still stay on "6", and will use ...[6] values to configure that interface, which is potentially damaging. Better safe than sorry: let's check for this overflow and do nothing, instead of potentially doing the wrong thing. Signed-off-by: Robby Workman <rworkman@slackware.com>
* rc.inet1: Avoid code duplication by combining case/esacJakub Jankowski2017-11-211-30/+9
| | | | | | | | | | | | | | There's no reason to have separate cases for start) ... ;; up) ... ;; if they both have the same bodies. Let's combine such cases to avoid code duplication. Also, this is that one time where quoting isn't really needed :) Signed-off-by: Robby Workman <rworkman@slackware.com>
* rc.inet1: Add missing 'dev' keywordJakub Jankowski2017-11-211-4/+4
| | | | | | Also, replace "ip addr" with "ip address" for consistency Signed-off-by: Robby Workman <rworkman@slackware.com>
* rc.inet1: Remove unnecessary $(echo)Jakub Jankowski2017-11-211-4/+4
| | | | Signed-off-by: Robby Workman <rworkman@slackware.com>
* rc.inet1*: Simplify virtif_* code, add note to example configJakub Jankowski2017-11-212-16/+11
| | | | | | | | | | | | | There is a built-in way of counting the number of elements in an array: ${#arrayname[@]}. Use this, and the fact that "for i in $(seq 0 -1)" will not evaluate body of the loop even once, to simplify code and get rid of virtif_determine(). While at it, add a note to rc.inet1.conf mentioning the necessity of keeping array indexes monotonically increasing (from 0), otherwise the code will fail (so would the original code). Signed-off-by: Robby Workman <rworkman@slackware.com>
* rc.inet1: Test for loopback being "state UNKNOWN" too.Jakub Jankowski2017-11-211-1/+1
| | | | | | | | On my system "ip link set dev lo up" does not put it into UP state, but rather UNKNOWN. So test for either when trying to decide whether we need to set it up. Signed-off-by: Robby Workman <rworkman@slackware.com>
* rc.inet1: Use simple test (-n/-z) for non-empty/emptyJakub Jankowski2017-11-211-13/+13
| | | | | | | | sh has built-in test for non-empty (-n) and empty (-z) variables, so let's use them. Also, quote variables, especially in tests (if [ ]) and array expansion (${foo[@]}). Signed-off-by: Robby Workman <rworkman@slackware.com>
* rc.inet1: Replace [ test1 -a test2 ] with [ test1 ] && [ test2 ]Jakub Jankowski2017-11-211-3/+3
| | | | | | | | | Conditional expression like [ test1 -a test2 ] should be avoided as it is not well defined. See http://pubs.opengroup.org/onlinepubs/9699919799/utilities/test.html#tag_20_128_16 and https://github.com/koalaman/shellcheck/wiki/SC2166 Signed-off-by: Robby Workman <rworkman@slackware.com>
* rc.inet1*: Fix DHCP timeoutcurrent-20171121Robby Workman2017-11-213-7/+7
| | | | | This was accidentally changed from 15 seconds back to the 14.2 default of 10 seconds. Thanks to Eduard Rozenberg.
* netconfig: Add /sbin/netconfigcurrent-20171117Robby Workman2017-11-171-0/+706
|
* rc.inet1: Remove ipmask usage (unnecessary now)Robby Workman2017-11-121-3/+0
|
* rc.inet1.8: Minor fixupsRobby Workman2017-10-071-3/+4
|
* rc.inet1.conf.5: Update for IPALIASES, bridge, and virtual IFsRobby Workman2017-10-071-7/+57
| | | | There are a few other fixups for previously existing stuff too.
* manpages/*: Add manual pages for network-scriptsRobby Workman2017-10-072-0/+312
|
* rc.inet1.conf: Clarify comment wrt bridge interfacesRobby Workman2017-10-071-1/+1
|
* Merge branch 'ipaliases'Robby Workman2017-07-143-0/+32
|\
| * README: Note current status of branchRobby Workman2017-07-141-2/+2
| |
| * rc.inet1.*: Added support for IP address aliasesRobby Workman2017-07-142-0/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | This is accomplished with a new parameter in rc.inet1.conf: IPALIASES[$index]="space separated list of addresses" The aliases are added with a /32 mask and a label of ${interface}:{$number} to remain compatible with net-tools. IP address aliases are supported for both statically configured interfaces and dhcp-configured interfaces, although they'll only be configured for dhcp if the dhcpcd call succeeds. * note: I don't what happens if dhcpcd gets IPv4LL address
* | README: noted current statusRobby Workman2017-07-131-3/+5
| |
* | Merge branch 'virtual-ifs'Robby Workman2017-07-133-9/+47
|\|
| * rc.inet1: Use bash builtin arithmetic instead of external exprRobby Workman2017-07-131-1/+1
| |
| * README: Noted status of virtual-ifs branchRobby Workman2017-07-121-25/+8
| |
| * rc.inet1: fix index number usage in virtual interface setup/destroyRobby Workman2017-07-121-1/+1
| |
* | README: Noted status of master branchRobby Workman2017-07-121-23/+7
| |
| * rc.inet1{,.conf}: Allow create/destroy of virtual tun/tap devicesRobby Workman2017-07-122-0/+44
|/
* rc.inet1: Fix bridge teardown codeRobby Workman2017-07-121-3/+3
|
* README: Additional notesRobby Workman2017-07-111-0/+26
|
* rc.inet1: Bugfixes...Robby Workman2017-07-111-4/+4
|
* rc.inet1: Replace backticks with $()Robby Workman2017-07-111-9/+9
|
* rc.wireless.conf: Replace ifconfig MAC lookup with "ip link show"Robby Workman2017-07-111-1/+2
|
* rc.wireless: Migrate HWADDR getting from ifconfig to iproute2Robby Workman2017-07-111-1/+1
| | | | TODO: Someone fluent with sed needs to review this...
* rc.wireless: Migrate ifconfig --> ip link set dev $IFRobby Workman2017-07-111-1/+1
|
* rc.inet1: Migrate bridge setup/teardown to iproute2 from brctlRobby Workman2017-07-111-4/+13
| | | | This should complete f7533fca7e82 and eliminate the TODO