* [PATCH libdrm] modetest: Fix to check return value of asprintf() [not found] <CGME20180110021623epcas1p2e9aaf571b99cd306a066bdc80822a84a@epcas1p2.samsung.com> @ 2018-01-10 2:16 ` Seung-Woo Kim 2018-01-22 12:09 ` Eric Engestrom 0 siblings, 1 reply; 3+ messages in thread From: Seung-Woo Kim @ 2018-01-10 2:16 UTC (permalink / raw) To: dri-devel; +Cc: sw0312.kim, emil.l.velikov There is warning about ignoring return value of 'asprintf'. Fix to check return value of asprintf(). Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com> --- tests/modetest/modetest.c | 5 ++++- 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/tests/modetest/modetest.c b/tests/modetest/modetest.c index 62d9332..0773bd0 100644 --- a/tests/modetest/modetest.c +++ b/tests/modetest/modetest.c @@ -656,10 +656,13 @@ static struct resources *get_resources(struct device *dev) for (i = 0; i < res->res->count_connectors; i++) { struct connector *connector = &res->connectors[i]; drmModeConnector *conn = connector->connector; + int num; - asprintf(&connector->name, "%s-%u", + num = asprintf(&connector->name, "%s-%u", util_lookup_connector_type_name(conn->connector_type), conn->connector_type_id); + if (num < 0) + goto error; } #define get_properties(_res, __res, type, Type) \ -- 1.7.4.1 _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH libdrm] modetest: Fix to check return value of asprintf() 2018-01-10 2:16 ` [PATCH libdrm] modetest: Fix to check return value of asprintf() Seung-Woo Kim @ 2018-01-22 12:09 ` Eric Engestrom 2018-01-23 0:29 ` Seung-Woo Kim 0 siblings, 1 reply; 3+ messages in thread From: Eric Engestrom @ 2018-01-22 12:09 UTC (permalink / raw) To: Seung-Woo Kim; +Cc: emil.l.velikov, dri-devel On Wednesday, 2018-01-10 11:16:41 +0900, Seung-Woo Kim wrote: > There is warning about ignoring return value of 'asprintf'. Fix to > check return value of asprintf(). > > Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com> Reviewed-by: Eric Engestrom <eric.engestrom@imgtec.com> Do you have commit access or would you like me to push it for you? > --- > tests/modetest/modetest.c | 5 ++++- > 1 files changed, 4 insertions(+), 1 deletions(-) > > diff --git a/tests/modetest/modetest.c b/tests/modetest/modetest.c > index 62d9332..0773bd0 100644 > --- a/tests/modetest/modetest.c > +++ b/tests/modetest/modetest.c > @@ -656,10 +656,13 @@ static struct resources *get_resources(struct device *dev) > for (i = 0; i < res->res->count_connectors; i++) { > struct connector *connector = &res->connectors[i]; > drmModeConnector *conn = connector->connector; > + int num; > > - asprintf(&connector->name, "%s-%u", > + num = asprintf(&connector->name, "%s-%u", > util_lookup_connector_type_name(conn->connector_type), > conn->connector_type_id); > + if (num < 0) > + goto error; > } > > #define get_properties(_res, __res, type, Type) \ > -- > 1.7.4.1 > _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH libdrm] modetest: Fix to check return value of asprintf() 2018-01-22 12:09 ` Eric Engestrom @ 2018-01-23 0:29 ` Seung-Woo Kim 0 siblings, 0 replies; 3+ messages in thread From: Seung-Woo Kim @ 2018-01-23 0:29 UTC (permalink / raw) To: Eric Engestrom; +Cc: Seung-Woo Kim, emil.l.velikov, dri-devel Hello, On 2018년 01월 22일 21:09, Eric Engestrom wrote: > On Wednesday, 2018-01-10 11:16:41 +0900, Seung-Woo Kim wrote: >> There is warning about ignoring return value of 'asprintf'. Fix to >> check return value of asprintf(). >> >> Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com> > > Reviewed-by: Eric Engestrom <eric.engestrom@imgtec.com> > > Do you have commit access or would you like me to push it for you? Thanks for reviewing. I don't have access permission, so please push it. Best Regards, - Seung-Woo Kim > >> --- >> tests/modetest/modetest.c | 5 ++++- >> 1 files changed, 4 insertions(+), 1 deletions(-) >> >> diff --git a/tests/modetest/modetest.c b/tests/modetest/modetest.c >> index 62d9332..0773bd0 100644 >> --- a/tests/modetest/modetest.c >> +++ b/tests/modetest/modetest.c >> @@ -656,10 +656,13 @@ static struct resources *get_resources(struct device *dev) >> for (i = 0; i < res->res->count_connectors; i++) { >> struct connector *connector = &res->connectors[i]; >> drmModeConnector *conn = connector->connector; >> + int num; >> >> - asprintf(&connector->name, "%s-%u", >> + num = asprintf(&connector->name, "%s-%u", >> util_lookup_connector_type_name(conn->connector_type), >> conn->connector_type_id); >> + if (num < 0) >> + goto error; >> } >> >> #define get_properties(_res, __res, type, Type) \ >> -- >> 1.7.4.1 >> > > > -- Seung-Woo Kim Samsung Research -- _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2018-01-23 0:29 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <CGME20180110021623epcas1p2e9aaf571b99cd306a066bdc80822a84a@epcas1p2.samsung.com>
2018-01-10 2:16 ` [PATCH libdrm] modetest: Fix to check return value of asprintf() Seung-Woo Kim
2018-01-22 12:09 ` Eric Engestrom
2018-01-23 0:29 ` Seung-Woo Kim
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.