diff options
author | Gabriel M. Beddingfield <gabrbedd@ti.com> | 2011-11-28 17:17:00 -0600 |
---|---|---|
committer | Gabriel M. Beddingfield <gabrbedd@ti.com> | 2012-02-08 16:58:27 -0600 |
commit | 3e3376a4b73e0475d6e1be9cf533ea4ae0674ee3 (patch) | |
tree | 12ddcb10ef5f64c3381ad00e7f31b8036fc0a8b8 /include | |
parent | 80085d470d189362ddb6dda9bba6ee05fe7c84c6 (diff) |
tinycap, tinyplay: Check *argv before dereferencing.
In several places, argv is incremented and *argv is dereferenced
without checking to see if it is valid to do so. This could lead to a
buffer overrun if the user provides invalid parameters.
This patch generally changes this:
if (strcmp(*argv, "-r") == 0) {
argv++;
rate = atoi(*argv);
}
argv++;
To this:
if (strcmp(*argv, "-r") == 0) {
argv++;
if (*argv)
rate = atoi(*argv);
}
if (*argv)
argv++;
Signed-off-by: Gabriel M. Beddingfield <gabrbedd@ti.com>
Diffstat (limited to 'include')
0 files changed, 0 insertions, 0 deletions