From: Zhengwang Ruan <ruan.zhengwang@gmail.com>
To: Greg KH <gregkh@linuxfoundation.org>
Cc: arve@android.com, chris+android@zenthought.org,
devel@driverdev.osuosl.org, kernel-janitors@vger.kernel.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH 2/2 v2] Staging:android: Initialise buffer and buffer_size before used
Date: Wed, 07 Mar 2012 14:16:20 +0800 [thread overview]
Message-ID: <4F56FD34.1090103@gmail.com> (raw)
In-Reply-To: <20120307054616.GA7775@kroah.com>
-------- Original Message --------
From: Greg KH
Sent: 2012年03月06日 星期二 21时46分16秒
To: Zhengwang Ruan
Subject: Re: [PATCH 2/2 v2] Staging:android: Initialise buffer and
buffer_size before used
> On Wed, Mar 07, 2012 at 10:36:58AM +0800, Zhengwang Ruan wrote:
>> GCC warns 'buffer' and 'buffer_size' are used with being uninitialized, and
>> 'buffer' is used as returned value, and 'buffer_size' is initialized using
>> uninitialized_var to clear warning.
> Doesn't say that for me at all, what version of gcc are you using? I'm
> using 4.6.2 here, and if you are using an older version, then please
> upgrade to one smarter than the one you are using.
>
> As you are trying to paper over gcc issues, I can't take this patch,
> sorry.
>
Ok, I will make another try after updating GCC to 4.6. Thank you for
your effort!
Thanks,
-Zhengwang
> greg k-h
>
next prev parent reply other threads:[~2012-03-07 6:16 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-03-07 2:36 [PATCH 1/2 v2] Staging:android: Change type for binder_debug_no_lock switch to bool Zhengwang Ruan
2012-03-07 2:36 ` [PATCH 2/2 v2] Staging:android: Initialise buffer and buffer_size before used Zhengwang Ruan
2012-03-07 4:01 ` Arve Hjønnevåg
2012-03-07 4:58 ` Zhengwang Ruan
2012-03-07 5:46 ` Greg KH
2012-03-07 6:16 ` Zhengwang Ruan [this message]
2012-03-07 6:00 ` Dan Carpenter
2012-03-07 6:01 ` Zhengwang Ruan
2012-03-07 6:20 ` Dan Carpenter
2012-03-07 6:25 ` Zhengwang Ruan
-- strict thread matches above, loose matches on Subject: below --
2012-03-07 2:49 [PATCH 1/2 v2] Staging:android: Change type for binder_debug_no_lock switch to bool ruan.zhengwang
2012-03-07 2:49 ` [PATCH 2/2 v2] Staging:android: Initialise buffer and buffer_size before used ruan.zhengwang
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=4F56FD34.1090103@gmail.com \
--to=ruan.zhengwang@gmail.com \
--cc=arve@android.com \
--cc=chris+android@zenthought.org \
--cc=devel@driverdev.osuosl.org \
--cc=gregkh@linuxfoundation.org \
--cc=kernel-janitors@vger.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: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox