summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorYu Watanabe <watanabe.yu+github@gmail.com>2020-11-09 16:41:17 +0900
committerYu Watanabe <watanabe.yu+github@gmail.com>2020-11-10 00:04:52 +0900
commitdd1b187075485414fc83c0d5615d78f4484ba712 (patch)
treeead4be7ca792fa6eca518cf0db36f057a9deb4df /src
parentac138551cee37ef277739809ab4ed87b3a31f2f9 (diff)
downloadsystemd-dd1b187075485414fc83c0d5615d78f4484ba712.tar.gz
sd-radv: refuse to set preference except medium when router lifetime is zero
Diffstat (limited to 'src')
-rw-r--r--src/libsystemd-network/sd-radv.c9
1 files changed, 6 insertions, 3 deletions
diff --git a/src/libsystemd-network/sd-radv.c b/src/libsystemd-network/sd-radv.c
index 5ad89ae7de..8beb845d79 100644
--- a/src/libsystemd-network/sd-radv.c
+++ b/src/libsystemd-network/sd-radv.c
@@ -506,17 +506,20 @@ _public_ int sd_radv_set_other_information(sd_radv *ra, int other) {
}
_public_ int sd_radv_set_preference(sd_radv *ra, unsigned preference) {
- int r = 0;
-
assert_return(ra, -EINVAL);
assert_return(IN_SET(preference,
SD_NDISC_PREFERENCE_LOW,
SD_NDISC_PREFERENCE_MEDIUM,
SD_NDISC_PREFERENCE_HIGH), -EINVAL);
+ /* RFC 4191, Section 2.2, "...If the Router Lifetime is zero, the preference value MUST be set
+ * to (00) by the sender..." */
+ if (ra->lifetime == 0 && preference != SD_NDISC_PREFERENCE_MEDIUM)
+ return -EINVAL;
+
ra->flags = (ra->flags & ~(0x3 << 3)) | (preference << 3);
- return r;
+ return 0;
}
_public_ int sd_radv_add_prefix(sd_radv *ra, sd_radv_prefix *p, int dynamic) {