From: Rui Miguel Silva <rmfrfs@gmail.com> To: driverdev-devel@linuxdriverproject.org Cc: Dan Carpenter <dan.carpenter@oracle.com>, devel@driverdev.osuosl.org, greybus-dev@lists.linaro.org, elder@kernel.org, johan@kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 0/9] staging: move greybus core out of staging Date: Tue, 27 Aug 2019 15:30:21 +0100 [thread overview] Message-ID: <m3d0gqisua.fsf@gmail.com> (raw) In-Reply-To: <20190827134557.GA25038@kroah.com> Hi, On Tue 27 Aug 2019 at 14:45, Greg Kroah-Hartman wrote: > On Tue, Aug 27, 2019 at 04:36:11PM +0300, Dan Carpenter wrote: >> I can't compile greybus so it's hard to run Smatch on it... I have a >> Smatch thing which ignores missing includes and just tries its best. >> It mostly generates garbage output but a couple of these look like >> potential issues: > > Why can't you compile the code? > I think we are missing includes in some of the greybus header files. > >> drivers/staging/greybus/operation.c:379 gb_operation_message_alloc() warn: check 'message_size' for integer overflows 'kzalloc()' > > That should be checked on line 368, right? > >> drivers/staging/greybus/light.c:1256 gb_lights_request_handler() warn: 'light->channels' double freed >> drivers/staging/greybus/light.c:1256 gb_lights_request_handler() warn: 'light->name' double freed > > I don't understand this warning, how are these potentially double freed? > > And the light.c file isn't moving out of drivers/staging/ just yet :) > I will take a look at this also. Cheers, Rui
WARNING: multiple messages have this Message-ID (diff)
From: Rui Miguel Silva <rmfrfs@gmail.com> To: driverdev-devel@linuxdriverproject.org Cc: devel@driverdev.osuosl.org, elder@kernel.org, johan@kernel.org, linux-kernel@vger.kernel.org, greybus-dev@lists.linaro.org, Dan Carpenter <dan.carpenter@oracle.com> Subject: Re: [PATCH 0/9] staging: move greybus core out of staging Date: Tue, 27 Aug 2019 15:30:21 +0100 [thread overview] Message-ID: <m3d0gqisua.fsf@gmail.com> (raw) In-Reply-To: <20190827134557.GA25038@kroah.com> Hi, On Tue 27 Aug 2019 at 14:45, Greg Kroah-Hartman wrote: > On Tue, Aug 27, 2019 at 04:36:11PM +0300, Dan Carpenter wrote: >> I can't compile greybus so it's hard to run Smatch on it... I have a >> Smatch thing which ignores missing includes and just tries its best. >> It mostly generates garbage output but a couple of these look like >> potential issues: > > Why can't you compile the code? > I think we are missing includes in some of the greybus header files. > >> drivers/staging/greybus/operation.c:379 gb_operation_message_alloc() warn: check 'message_size' for integer overflows 'kzalloc()' > > That should be checked on line 368, right? > >> drivers/staging/greybus/light.c:1256 gb_lights_request_handler() warn: 'light->channels' double freed >> drivers/staging/greybus/light.c:1256 gb_lights_request_handler() warn: 'light->name' double freed > > I don't understand this warning, how are these potentially double freed? > > And the light.c file isn't moving out of drivers/staging/ just yet :) > I will take a look at this also. Cheers, Rui _______________________________________________ devel mailing list devel@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
next prev parent reply other threads:[~2019-08-27 14:30 UTC|newest] Thread overview: 41+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-08-25 5:54 [PATCH 0/9] staging: move greybus core out of staging Greg Kroah-Hartman 2019-08-25 5:54 ` [PATCH 1/9] staging: greybus: fix up SPDX comment in .h files Greg Kroah-Hartman 2019-08-26 5:51 ` Viresh Kumar 2019-08-27 7:58 ` Greg Kroah-Hartman 2019-08-27 9:20 ` Gao Xiang 2019-08-27 15:43 ` Greg Kroah-Hartman 2019-08-27 16:01 ` Gao Xiang 2019-08-25 5:54 ` [PATCH 2/9] staging: greybus: remove license "boilerplate" Greg Kroah-Hartman 2019-08-26 5:52 ` Viresh Kumar 2019-08-27 15:37 ` Mark Greer 2019-08-25 5:54 ` [PATCH 3/9] staging: greybus: hd: Fix up some alignment checkpatch issues Greg Kroah-Hartman 2019-08-26 5:57 ` [greybus-dev] " Viresh Kumar 2019-08-25 5:54 ` [PATCH 4/9] staging: greybus: manifest: " Greg Kroah-Hartman 2019-08-26 6:11 ` [greybus-dev] " Viresh Kumar 2019-08-25 5:54 ` [PATCH 5/9] staging: greybus: log: " Greg Kroah-Hartman 2019-08-26 6:12 ` [greybus-dev] " Viresh Kumar 2019-08-25 5:54 ` [PATCH 6/9] staging: greybus: loopback: " Greg Kroah-Hartman 2019-08-26 6:12 ` [greybus-dev] " Viresh Kumar 2019-08-25 5:54 ` [PATCH 7/9] staging: greybus: move core include files to include/linux/greybus/ Greg Kroah-Hartman 2019-08-26 5:55 ` Viresh Kumar 2019-08-27 15:38 ` Mark Greer 2019-08-25 5:54 ` [PATCH 8/9] staging: greybus: move the greybus core to drivers/greybus Greg Kroah-Hartman 2019-08-26 6:13 ` [greybus-dev] " Viresh Kumar 2019-08-25 5:54 ` [PATCH 9/9] staging: greybus: move es2 to drivers/greybus/ Greg Kroah-Hartman 2019-08-26 6:14 ` [greybus-dev] " Viresh Kumar 2019-08-27 10:43 ` [greybus-dev] [PATCH 0/9] staging: move greybus core out of staging Alex Elder 2019-08-27 13:36 ` Dan Carpenter 2019-08-27 13:45 ` Greg Kroah-Hartman 2019-08-27 14:30 ` Rui Miguel Silva [this message] 2019-08-27 14:30 ` Rui Miguel Silva 2019-08-27 15:43 ` Greg KH 2019-08-27 15:43 ` Greg KH 2019-08-27 15:57 ` Rui Miguel Silva 2019-08-27 15:57 ` Rui Miguel Silva 2019-08-27 16:59 ` Greg KH 2019-08-27 16:59 ` Greg KH 2019-08-27 14:37 ` Dan Carpenter 2019-09-01 17:23 ` Pavel Machek 2019-09-01 17:23 ` Pavel Machek 2019-09-02 6:31 ` Greg Kroah-Hartman 2019-09-02 6:31 ` Greg Kroah-Hartman
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=m3d0gqisua.fsf@gmail.com \ --to=rmfrfs@gmail.com \ --cc=dan.carpenter@oracle.com \ --cc=devel@driverdev.osuosl.org \ --cc=driverdev-devel@linuxdriverproject.org \ --cc=elder@kernel.org \ --cc=greybus-dev@lists.linaro.org \ --cc=johan@kernel.org \ --cc=linux-kernel@vger.kernel.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
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.