Skip to content

L
linux
Commit 27d5a87c authored by Hans Verkuil's avatar Hans Verkuil Committed by Mauro Carvalho Chehab
Browse files
Options

[media] v4l2-ioctl: add precision when printing names 

Never print more than the size of the buffer containing the name.
Signed-off-by: default avatarHans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@redhat.com>
parent 314527ac
Hide whitespace changes
Inline Side-by-side
Showing with 34 additions and 26 deletions
drivers/media/v4l2-core/v4l2-ioctl.c
View file @ 27d5a87c
...@@ -167,9 +167,11 @@ static void v4l_print_querycap(const void *arg, bool write_only) ...@@ -167,9 +167,11 @@ static void v4l_print_querycap(const void *arg, bool write_only)
{ {
const struct v4l2_capability *p = arg; const struct v4l2_capability *p = arg;
pr_cont("driver=%s, card=%s, bus=%s, version=0x%08x, " pr_cont("driver=%.*s, card=%.*s, bus=%.*s, version=0x%08x, "
"capabilities=0x%08x, device_caps=0x%08x\n", "capabilities=0x%08x, device_caps=0x%08x\n",
p->driver, p->card, p->bus_info, (int)sizeof(p->driver), p->driver,
(int)sizeof(p->card), p->card,
(int)sizeof(p->bus_info), p->bus_info,
p->version, p->capabilities, p->device_caps); p->version, p->capabilities, p->device_caps);
} }
...@@ -177,20 +179,21 @@ static void v4l_print_enuminput(const void *arg, bool write_only) ...@@ -177,20 +179,21 @@ static void v4l_print_enuminput(const void *arg, bool write_only)
{ {
const struct v4l2_input *p = arg; const struct v4l2_input *p = arg;
pr_cont("index=%u, name=%s, type=%u, audioset=0x%x, tuner=%u, " pr_cont("index=%u, name=%.*s, type=%u, audioset=0x%x, tuner=%u, "
"std=0x%08Lx, status=0x%x, capabilities=0x%x\n", "std=0x%08Lx, status=0x%x, capabilities=0x%x\n",
p->index, p->name, p->type, p->audioset, p->tuner, p->index, (int)sizeof(p->name), p->name, p->type, p->audioset,
(unsigned long long)p->std, p->status, p->capabilities); p->tuner, (unsigned long long)p->std, p->status,
p->capabilities);
} }
static void v4l_print_enumoutput(const void *arg, bool write_only) static void v4l_print_enumoutput(const void *arg, bool write_only)
{ {
const struct v4l2_output *p = arg; const struct v4l2_output *p = arg;
pr_cont("index=%u, name=%s, type=%u, audioset=0x%x, " pr_cont("index=%u, name=%.*s, type=%u, audioset=0x%x, "
"modulator=%u, std=0x%08Lx, capabilities=0x%x\n", "modulator=%u, std=0x%08Lx, capabilities=0x%x\n",
p->index, p->name, p->type, p->audioset, p->modulator, p->index, (int)sizeof(p->name), p->name, p->type, p->audioset,
(unsigned long long)p->std, p->capabilities); p->modulator, (unsigned long long)p->std, p->capabilities);
} }
static void v4l_print_audio(const void *arg, bool write_only) static void v4l_print_audio(const void *arg, bool write_only)
...@@ -200,8 +203,9 @@ static void v4l_print_audio(const void *arg, bool write_only) ...@@ -200,8 +203,9 @@ static void v4l_print_audio(const void *arg, bool write_only)
if (write_only) if (write_only)
pr_cont("index=%u, mode=0x%x\n", p->index, p->mode); pr_cont("index=%u, mode=0x%x\n", p->index, p->mode);
else else
pr_cont("index=%u, name=%s, capability=0x%x, mode=0x%x\n", pr_cont("index=%u, name=%.*s, capability=0x%x, mode=0x%x\n",
p->index, p->name, p->capability, p->mode); p->index, (int)sizeof(p->name), p->name,
p->capability, p->mode);
} }
static void v4l_print_audioout(const void *arg, bool write_only) static void v4l_print_audioout(const void *arg, bool write_only)
...@@ -211,21 +215,22 @@ static void v4l_print_audioout(const void *arg, bool write_only) ...@@ -211,21 +215,22 @@ static void v4l_print_audioout(const void *arg, bool write_only)
if (write_only) if (write_only)
pr_cont("index=%u\n", p->index); pr_cont("index=%u\n", p->index);
else else
pr_cont("index=%u, name=%s, capability=0x%x, mode=0x%x\n", pr_cont("index=%u, name=%.*s, capability=0x%x, mode=0x%x\n",
p->index, p->name, p->capability, p->mode); p->index, (int)sizeof(p->name), p->name,
p->capability, p->mode);
} }
static void v4l_print_fmtdesc(const void *arg, bool write_only) static void v4l_print_fmtdesc(const void *arg, bool write_only)
{ {
const struct v4l2_fmtdesc *p = arg; const struct v4l2_fmtdesc *p = arg;
pr_cont("index=%u, type=%s, flags=0x%x, pixelformat=%c%c%c%c, description='%s'\n", pr_cont("index=%u, type=%s, flags=0x%x, pixelformat=%c%c%c%c, description='%.*s'\n",
p->index, prt_names(p->type, v4l2_type_names), p->index, prt_names(p->type, v4l2_type_names),
p->flags, (p->pixelformat & 0xff), p->flags, (p->pixelformat & 0xff),
(p->pixelformat >> 8) & 0xff, (p->pixelformat >> 8) & 0xff,
(p->pixelformat >> 16) & 0xff, (p->pixelformat >> 16) & 0xff,
(p->pixelformat >> 24) & 0xff, (p->pixelformat >> 24) & 0xff,
p->description); (int)sizeof(p->description), p->description);
} }
static void v4l_print_format(const void *arg, bool write_only) static void v4l_print_format(const void *arg, bool write_only)
...@@ -348,9 +353,9 @@ static void v4l_print_modulator(const void *arg, bool write_only) ...@@ -348,9 +353,9 @@ static void v4l_print_modulator(const void *arg, bool write_only)
if (write_only) if (write_only)
pr_cont("index=%u, txsubchans=0x%x", p->index, p->txsubchans); pr_cont("index=%u, txsubchans=0x%x", p->index, p->txsubchans);
else else
pr_cont("index=%u, name=%s, capability=0x%x, " pr_cont("index=%u, name=%.*s, capability=0x%x, "
"rangelow=%u, rangehigh=%u, txsubchans=0x%x\n", "rangelow=%u, rangehigh=%u, txsubchans=0x%x\n",
p->index, p->name, p->capability, p->index, (int)sizeof(p->name), p->name, p->capability,
p->rangelow, p->rangehigh, p->txsubchans); p->rangelow, p->rangehigh, p->txsubchans);
} }
...@@ -361,10 +366,10 @@ static void v4l_print_tuner(const void *arg, bool write_only) ...@@ -361,10 +366,10 @@ static void v4l_print_tuner(const void *arg, bool write_only)
if (write_only) if (write_only)
pr_cont("index=%u, audmode=%u\n", p->index, p->audmode); pr_cont("index=%u, audmode=%u\n", p->index, p->audmode);
else else
pr_cont("index=%u, name=%s, type=%u, capability=0x%x, " pr_cont("index=%u, name=%.*s, type=%u, capability=0x%x, "
"rangelow=%u, rangehigh=%u, signal=%u, afc=%d, " "rangelow=%u, rangehigh=%u, signal=%u, afc=%d, "
"rxsubchans=0x%x, audmode=%u\n", "rxsubchans=0x%x, audmode=%u\n",
p->index, p->name, p->type, p->index, (int)sizeof(p->name), p->name, p->type,
p->capability, p->rangelow, p->capability, p->rangelow,
p->rangehigh, p->signal, p->afc, p->rangehigh, p->signal, p->afc,
p->rxsubchans, p->audmode); p->rxsubchans, p->audmode);
...@@ -382,9 +387,9 @@ static void v4l_print_standard(const void *arg, bool write_only) ...@@ -382,9 +387,9 @@ static void v4l_print_standard(const void *arg, bool write_only)
{ {
const struct v4l2_standard *p = arg; const struct v4l2_standard *p = arg;
pr_cont("index=%u, id=0x%Lx, name=%s, fps=%u/%u, " pr_cont("index=%u, id=0x%Lx, name=%.*s, fps=%u/%u, "
"framelines=%u\n", p->index, "framelines=%u\n", p->index,
(unsigned long long)p->id, p->name, (unsigned long long)p->id, (int)sizeof(p->name), p->name,
p->frameperiod.numerator, p->frameperiod.numerator,
p->frameperiod.denominator, p->frameperiod.denominator,
p->framelines); p->framelines);
...@@ -504,9 +509,9 @@ static void v4l_print_queryctrl(const void *arg, bool write_only) ...@@ -504,9 +509,9 @@ static void v4l_print_queryctrl(const void *arg, bool write_only)
{ {
const struct v4l2_queryctrl *p = arg; const struct v4l2_queryctrl *p = arg;
pr_cont("id=0x%x, type=%d, name=%s, min/max=%d/%d, " pr_cont("id=0x%x, type=%d, name=%.*s, min/max=%d/%d, "
"step=%d, default=%d, flags=0x%08x\n", "step=%d, default=%d, flags=0x%08x\n",
p->id, p->type, p->name, p->id, p->type, (int)sizeof(p->name), p->name,
p->minimum, p->maximum, p->minimum, p->maximum,
p->step, p->default_value, p->flags); p->step, p->default_value, p->flags);
} }
...@@ -623,7 +628,8 @@ static void v4l_print_dbg_chip_ident(const void *arg, bool write_only) ...@@ -623,7 +628,8 @@ static void v4l_print_dbg_chip_ident(const void *arg, bool write_only)
pr_cont("type=%u, ", p->match.type); pr_cont("type=%u, ", p->match.type);
if (p->match.type == V4L2_CHIP_MATCH_I2C_DRIVER) if (p->match.type == V4L2_CHIP_MATCH_I2C_DRIVER)
pr_cont("name=%s, ", p->match.name); pr_cont("name=%.*s, ",
(int)sizeof(p->match.name), p->match.name);
else else
pr_cont("addr=%u, ", p->match.addr); pr_cont("addr=%u, ", p->match.addr);
pr_cont("chip_ident=%u, revision=0x%x\n", pr_cont("chip_ident=%u, revision=0x%x\n",
...@@ -636,7 +642,8 @@ static void v4l_print_dbg_register(const void *arg, bool write_only) ...@@ -636,7 +642,8 @@ static void v4l_print_dbg_register(const void *arg, bool write_only)
pr_cont("type=%u, ", p->match.type); pr_cont("type=%u, ", p->match.type);
if (p->match.type == V4L2_CHIP_MATCH_I2C_DRIVER) if (p->match.type == V4L2_CHIP_MATCH_I2C_DRIVER)
pr_cont("name=%s, ", p->match.name); pr_cont("name=%.*s, ",
(int)sizeof(p->match.name), p->match.name);
else else
pr_cont("addr=%u, ", p->match.addr); pr_cont("addr=%u, ", p->match.addr);
pr_cont("reg=0x%llx, val=0x%llx\n", pr_cont("reg=0x%llx, val=0x%llx\n",
...@@ -647,8 +654,9 @@ static void v4l_print_dv_enum_presets(const void *arg, bool write_only) ...@@ -647,8 +654,9 @@ static void v4l_print_dv_enum_presets(const void *arg, bool write_only)
{ {
const struct v4l2_dv_enum_preset *p = arg; const struct v4l2_dv_enum_preset *p = arg;
pr_cont("index=%u, preset=%u, name=%s, width=%u, height=%u\n", pr_cont("index=%u, preset=%u, name=%.*s, width=%u, height=%u\n",
p->index, p->preset, p->name, p->width, p->height); p->index, p->preset,
(int)sizeof(p->name), p->name, p->width, p->height);
} }
static void v4l_print_dv_preset(const void *arg, bool write_only) static void v4l_print_dv_preset(const void *arg, bool write_only)
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment
GitLab Nexedi Edition | About GitLab | About Nexedi | 沪ICP备2021021310号-2 | 沪ICP备2021021310号-7