File 58343f79-pygrub-Properly-quote-results-when-ret... of Package xen.3679

Overview Repositories Revisions Requests Users Attributes Meta

File 58343f79-pygrub-Properly-quote-results-when-returning-them-to-the-caller.patch of Package xen.3679

Subject: pygrub: Properly quote results, when returning them to the caller:
From: Ian Jackson Ian.Jackson@eu.citrix.com Tue Nov 22 14:30:27 2016 +0100
Date: Tue Nov 22 14:30:27 2016 +0100:
Git: 8e7b84dd2a187edc74f44b69437734b8e4af9628

* When the caller wants sexpr output, use `repr()'
  This is what Xend expects.

The returned S-expressions are now escaped and quoted by Python,
  generally using '...'.  Previously kernel and ramdisk were unquoted
  and args was quoted with "..." but without proper escaping.  This
  change may break toolstacks which do not properly dequote the
  returned S-expressions.

* When the caller wants "simple" output, crash if the delimiter is
  contained in the returned value.

With --output-format=simple it does not seem like this could ever
  happen, because the bootloader config parsers all take line-based
  input from the various bootloader config files.

With --output-format=simple0, this can happen if the bootloader
  config file contains nul bytes.

This is CVE-2016-9379 and CVE-2016-9380 / XSA-198.

Signed-off-by: Ian Jackson <Ian.Jackson@eu.citrix.com>
Tested-by: Ian Jackson <Ian.Jackson@eu.citrix.com>
Reviewed-by: Andrew Cooper <andrew.cooper3@citrix.com>
master commit: 27e14d346ed6ff1c3a3cfc479507e62d133e92a9
master date: 2016-11-22 13:52:09 +0100

diff --git a/tools/pygrub/src/pygrub b/tools/pygrub/src/pygrub
index 00cf77e..f21fe18 100755
--- a/tools/pygrub/src/pygrub
+++ b/tools/pygrub/src/pygrub
@@ -722,14 +722,17 @@ def sniff_netware(fs, cfg):
     return cfg
 
 def format_sxp(kernel, ramdisk, args):
-    s = "linux (kernel %s)" % kernel
+    s = "linux (kernel %s)" % repr(kernel)
     if ramdisk:
-        s += "(ramdisk %s)" % ramdisk
+        s += "(ramdisk %s)" % repr(ramdisk)
     if args:
-        s += "(args \"%s\")" % args
+        s += "(args %s)" % repr(args)
     return s
                 
 def format_simple(kernel, ramdisk, args, sep):
+    for check in (kernel, ramdisk, args):
+        if check is not None and sep in check:
+            raise RuntimeError, "simple format cannot represent delimiter-containing value"
     s = ("kernel %s" % kernel) + sep
     if ramdisk:
         s += ("ramdisk %s" % ramdisk) + sep

Places

File 58343f79-pygrub-Properly-quote-results-when-returning-them-to-the-caller.patch of Package xen.3679

Places