code.ossystems Code Review - openembedded-core.git/blob

   1 From 81c90dc8f0aae3b65730409b1b615b5fa7280ebd Mon Sep 17 00:00:00 2001
   2 From: Olivier Fourdan <ofourdan@redhat.com>
   3 Date: Fri, 16 Jan 2015 20:08:59 +0100
   4 Subject: [PATCH] xkb: Don't swap XkbSetGeometry data in the input buffer
   5
   6 The XkbSetGeometry request embeds data which needs to be swapped when the
   7 server and the client have different endianess.
   8
   9 _XkbSetGeometry() invokes functions that swap these data directly in the
  10 input buffer.
  11
  12 However, ProcXkbSetGeometry() may call _XkbSetGeometry() more than once
  13 (if there is more than one keyboard), thus causing on swapped clients the
  14 same data to be swapped twice in memory, further causing a server crash
  15 because the strings lengths on the second time are way off bounds.
  16
  17 To allow _XkbSetGeometry() to run reliably more than once with swapped
  18 clients, do not swap the data in the buffer, use variables instead.
  19
  20 Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>
  21 Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
  22
  23 Upstream-Status: backport
  24
  25 Signed-off-by: Li Zhou <li.zhou@windriver.com>
  26 ---
  27  xkb/xkb.c |   35 +++++++++++++++++++----------------
  28  1 file changed, 19 insertions(+), 16 deletions(-)
  29
  30 diff --git a/xkb/xkb.c b/xkb/xkb.c
  31 index 15c7f34..b9a3ac4 100644
  32 --- a/xkb/xkb.c
  33 +++ b/xkb/xkb.c
  34 @@ -4961,14 +4961,13 @@ static char *
  35  _GetCountedString(char **wire_inout, Bool swap)
  36  {
  37      char *wire, *str;
  38 -    CARD16 len, *plen;
  39 +    CARD16 len;
  40
  41      wire = *wire_inout;
  42 -    plen = (CARD16 *) wire;
  43 +    len = *(CARD16 *) wire;
  44      if (swap) {
  45 -        swaps(plen);
  46 +        swaps(&len);
  47      }
  48 -    len = *plen;
  49      str = malloc(len + 1);
  50      if (str) {
  51          memcpy(str, &wire[2], len);
  52 @@ -4985,25 +4984,28 @@ _CheckSetDoodad(char **wire_inout,
  53  {
  54      char *wire;
  55      xkbDoodadWireDesc *dWire;
  56 +    xkbAnyDoodadWireDesc any;
  57 +    xkbTextDoodadWireDesc text;
  58      XkbDoodadPtr doodad;
  59
  60      dWire = (xkbDoodadWireDesc *) (*wire_inout);
  61 +    any = dWire->any;
  62      wire = (char *) &dWire[1];
  63      if (client->swapped) {
  64 -        swapl(&dWire->any.name);
  65 -        swaps(&dWire->any.top);
  66 -        swaps(&dWire->any.left);
  67 -        swaps(&dWire->any.angle);
  68 +        swapl(&any.name);
  69 +        swaps(&any.top);
  70 +        swaps(&any.left);
  71 +        swaps(&any.angle);
  72      }
  73      CHK_ATOM_ONLY(dWire->any.name);
  74 -    doodad = XkbAddGeomDoodad(geom, section, dWire->any.name);
  75 +    doodad = XkbAddGeomDoodad(geom, section, any.name);
  76      if (!doodad)
  77          return BadAlloc;
  78      doodad->any.type = dWire->any.type;
  79      doodad->any.priority = dWire->any.priority;
  80 -    doodad->any.top = dWire->any.top;
  81 -    doodad->any.left = dWire->any.left;
  82 -    doodad->any.angle = dWire->any.angle;
  83 +    doodad->any.top = any.top;
  84 +    doodad->any.left = any.left;
  85 +    doodad->any.angle = any.angle;
  86      switch (doodad->any.type) {
  87      case XkbOutlineDoodad:
  88      case XkbSolidDoodad:
  89 @@ -5026,12 +5028,13 @@ _CheckSetDoodad(char **wire_inout,
  90                                                dWire->text.colorNdx);
  91              return BadMatch;
  92          }
  93 +        text = dWire->text;
  94          if (client->swapped) {
  95 -            swaps(&dWire->text.width);
  96 -            swaps(&dWire->text.height);
  97 +            swaps(&text.width);
  98 +            swaps(&text.height);
  99          }
 100 -        doodad->text.width = dWire->text.width;
 101 -        doodad->text.height = dWire->text.height;
 102 +        doodad->text.width = text.width;
 103 +        doodad->text.height = text.height;
 104          doodad->text.color_ndx = dWire->text.colorNdx;
 105          doodad->text.text = _GetCountedString(&wire, client->swapped);
 106          doodad->text.font = _GetCountedString(&wire, client->swapped);
 107 --
 108 1.7.9.5
 109