~ubuntu-branches/ubuntu/maverick/xorg-server/maverick-security

« back to all changes in this revision

Viewing changes to hw/xwin/winvalargs.c

  • Committer: Bazaar Package Importer
  • Author(s): Christopher James Halse Rogers
  • Date: 2010-08-05 11:25:14 UTC
  • mfrom: (1.1.35 upstream) (0.1.14 experimental)
  • Revision ID: james.westby@ubuntu.com-20100805112514-q4efdgj3nblevos2
Tags: 2:1.8.99.905-1ubuntu1
* Merge from (unreleased) Debian experimental.  Remaining Ubuntu changes:
  - rules, control:
    + Disable SELinux, libaudit-dev is not in main yet (LP 406226).
      Drop libaudit-dev from build-deps.
  - rules: Enable xcsecurity (LP 247537).
  - local/xvfb-run*: Add correct docs about error codes (LP 328205)
  - rules: Add --with-extra-module-dir to support GL alternatives.
  - control: Xvfb depends on xauth, x11-xkb-utils. (LP 500102)
  - rules, local/64-xorg-xkb.rules: Don't use keyboard-configuration
    until it's available.
  - control: Update some versioned Breaks for Ubuntu versions.
  - debian/patches:
    + 100_rethrow_signals.patch:
      When aborting, re-raise signals for apport
    + 109_fix-swcursor-crash.patch:
      Avoid dereferencing null pointer while reloading cursors during
      resume. (LP 371405)
    + 111_armel-drv-fallbacks.patch:
      Add support for armel driver fallbacks.
    + 121_only_switch_vt_when_active.diff:
      Add a check to prevent the X server from changing the VT when killing
      GDM from the console.
    + 122_xext_fix_card32_overflow_in_xauth.patch:
      Fix server crash when “xauth generate” is called with large timeout.
    + 157_check_null_modes.patch, 162_null_crtc_in_rotation.patch,
      166_nullptr_xinerama_keyrepeat.patch, 167_nullptr_xisbread.patch
      169_mipointer_nullptr_checks.patch,
      172_cwgetbackingpicture_nullptr_check.patch:
      Fix various segfaults in xserver by checking pointers for NULL
      values before dereferencing them.
    + 165_man_xorg_conf_no_device_ident.patch
      Correct man page
    + 168_glibc_trace_to_stderr.patch:
      Report abort traces to stderr instead of terminal
    + 184_virtual_devices_autodetect.patch:
      Use vesa for qemu device, which is not supported by cirrus
    + 187_edid_quirk_hp_nc8430.patch:
      Quirk for another LPL monitor (LP 380009)
    + 188_default_primary_to_first_busid.patch:
      Pick the first device and carry on (LP 459512)
    + 189_xserver_1.5.0_bg_none_root.patch:
      Create a root window with no background.
    + 190_cache-xkbcomp_output_for_fast_start_up.patch:
      Cache keyboard settings.
    + 191-Xorg-add-an-extra-module-path.patch:
      Add support for the alternatives module path.
    + 197_xvfb-randr.patch:
      Adds xrandr support to xvfb. (LP 516123)
    + 198_nohwaccess.patch:
      Adds a -nohwaccess argument to make X not access the hardware
      ports directly.
    + 200_randr-null.patch:
      Clarify a pointer initialization.
* Update changelog entries for 1.8.1.902-1 which became 1.8.99.904-1
* Drop 196_xvfbscreeninit-handling.patch: it's semantically empty, and now 
  doesn't apply.  Merge remaining #include change into 197_xvfb-randr.patch
* New upstream version will start correctly when no outputs are connected,
  as long as the video driver can dynamically resize the framebuffer
  (true for all KMS drivers) (LP: #337889)
* New upstream version fixes crash on non-admin logout with KDE (LP: #569879)
* Refresh 111_armel-drv-fallbacks.patch to fix the build on armel

Show diffs side-by-side

added added

removed removed

Lines of Context:
40
40
 */
41
41
 
42
42
extern int                      g_iNumScreens;
43
 
extern winScreenInfo            g_ScreenInfo[];
 
43
extern winScreenInfo *          g_ScreenInfo;
44
44
extern Bool                     g_fXdmcpEnabled;
45
45
 
46
46
 
47
47
/*
48
 
 * Prototypes
 
48
 * Verify all screens have been explicitly specified
49
49
 */
50
 
 
51
 
Bool
52
 
winValidateArgs (void);
53
 
 
 
50
static BOOL
 
51
isEveryScreenExplicit(void)
 
52
{
 
53
  int i;
 
54
 
 
55
  for (i = 0; i < g_iNumScreens; i++)
 
56
    if (!g_ScreenInfo[i].fExplicitScreen)
 
57
      return FALSE;
 
58
 
 
59
  return TRUE;
 
60
}
54
61
 
55
62
/*
56
63
 * winValidateArgs - Look for invalid argument combinations
62
69
  int           i;
63
70
  int           iMaxConsecutiveScreen = 0;
64
71
  BOOL          fHasNormalScreen0 = FALSE;
 
72
  BOOL          fImplicitScreenFound = FALSE;
65
73
 
66
74
  /*
67
75
   * Check for a malformed set of -screen parameters.
70
78
   *    XWin -screen 0 -screen 2
71
79
   *    XWin -screen 1 -screen 2
72
80
   */
73
 
  for (i = 0; i < MAXSCREENS; i++)
74
 
    {
75
 
      if (g_ScreenInfo[i].fExplicitScreen)
76
 
        iMaxConsecutiveScreen = i + 1;
77
 
    }
78
 
  winErrorFVerb (2, "winValidateArgs - g_iNumScreens: %d "
79
 
                 "iMaxConsecutiveScreen: %d\n",
80
 
                 g_iNumScreens, iMaxConsecutiveScreen);
81
 
  if (g_iNumScreens < iMaxConsecutiveScreen)
 
81
  if (!isEveryScreenExplicit())
82
82
    {
83
83
      ErrorF ("winValidateArgs - Malformed set of screen parameter(s).  "
84
84
              "Screens must be specified consecutively starting with "
85
85
              "screen 0.  That is, you cannot have only a screen 1, nor "
86
86
              "could you have screen 0 and screen 2.  You instead must "
87
 
              "have screen 0, or screen 0 and screen 1, respectively.  Of "
88
 
              "you can specify as many screens as you want from 0 up to "
89
 
              "%d.\n", MAXSCREENS - 1);
 
87
              "have screen 0, or screen 0 and screen 1, respectively.  "
 
88
              "You can specify as many screens as you want.\n");
90
89
      return FALSE;
91
90
    }
92
91