Whamcloud - gitweb
LUDOC-496 nrs: Format of TBF rule name field is not defined 44/45144/3
authorEtienne AUJAMES <etienne.aujames@cea.fr>
Thu, 7 Oct 2021 08:12:55 +0000 (10:12 +0200)
committerAndreas Dilger <adilger@whamcloud.com>
Thu, 7 Oct 2021 16:39:11 +0000 (16:39 +0000)
Lustre manual should define the format (alphanumeric + '_') and length
(15 characters) of a TBF rule name field.

This is important because there is buffer overflow for the TBF rule name
(LU-15059): the length is not checked by ptlrpc.

Change-Id: I29efb7e5891c0fba308b683bf92b26e4fefa7695
Signed-off-by: Etienne AUJAMES <eaujames@ddn.com>
Reviewed-on: https://review.whamcloud.com/45144
Tested-by: jenkins <devops@whamcloud.com>
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
LustreTuning.xml

index a8bde00..9460749 100644 (file)
@@ -1636,10 +1636,13 @@ $ lctl set_param ost.OSS.ost_io.nrs_policies="tbf gid"</screen>
        <screen>lctl set_param x.x.x.nrs_tbf_rule=
 "[reg|hp] start <replaceable>rule_name</replaceable> <replaceable>arguments</replaceable>..."
        </screen>
-       <para>'<replaceable>rule_name</replaceable>' is a string of the TBF
-       policy rule's name and '<replaceable>arguments</replaceable>' is a
-       string to specify the detailed rule according to the different types.
-       </para>
+        <para>'<replaceable>rule_name</replaceable>' is a string up to 15
+          characters which identifies the TBF policy rule's name. Alphanumeric
+          characters and underscores are accepted (e.g: "test_rule_A1").
+        </para>
+        <para>'<replaceable>arguments</replaceable>' is a string to specify the
+          detailed rule according to the different types.
+        </para>
        <itemizedlist>
        <para>Next, the different types of TBF policies will be described.</para>
          <listitem>