diff options
author | Andy Ritger <aritger@nvidia.com> | 2013-05-02 01:20:27 -0700 |
---|---|---|
committer | Aaron Plattner <aplattner@nvidia.com> | 2013-05-06 12:45:51 -0700 |
commit | ace5e2b21b2a204f94c6452fa0d8f36699aec249 (patch) | |
tree | 2675b537e950b711fcf61cf72fa62b6e4092c871 | |
parent | eb17ad8adc9400f6ed252872f13ccf5551f9e2e9 (diff) |
xrandr: calloc XRRModeInfo's passed to libXrandr.
The "--newmode" commandline option initializes an XRRModeInfo and passes
it into XRRCreateMode(). calloc(3) it to avoid uninitialized fields.
For consistency, calloc(3) all the places where umode_t's (the wrapper
structure for XRRModeInfo) are allocated.
Signed-off-by: Andy Ritger <aritger@nvidia.com>
Tested-by: Nikhil Mahale <nmahale@nvidia.com>
Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
-rw-r--r-- | xrandr.c | 8 |
1 files changed, 4 insertions, 4 deletions
@@ -2924,7 +2924,7 @@ main (int argc, char **argv) } if (!strcmp ("--newmode", argv[i])) { - umode_t *m = malloc (sizeof (umode_t)); + umode_t *m = calloc (1, sizeof (umode_t)); double clock; ++i; @@ -2966,7 +2966,7 @@ main (int argc, char **argv) } if (!strcmp ("--rmmode", argv[i])) { - umode_t *m = malloc (sizeof (umode_t)); + umode_t *m = calloc (1, sizeof (umode_t)); if (++i >= argc) argerr ("%s requires an argument\n", argv[i-1]); set_name (&m->name, argv[i], name_string|name_xid); @@ -2979,7 +2979,7 @@ main (int argc, char **argv) } if (!strcmp ("--addmode", argv[i])) { - umode_t *m = malloc (sizeof (umode_t)); + umode_t *m = calloc (1, sizeof (umode_t)); if (i+2 >= argc) argerr ("%s requires two arguments\n", argv[i]); set_name (&m->output, argv[++i], name_string|name_xid); @@ -2993,7 +2993,7 @@ main (int argc, char **argv) } if (!strcmp ("--delmode", argv[i])) { - umode_t *m = malloc (sizeof (umode_t)); + umode_t *m = calloc (1, sizeof (umode_t)); if (i+2 >= argc) argerr ("%s requires two arguments\n", argv[i]); set_name (&m->output, argv[++i], name_string|name_xid); |