From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from intranet.asianux.com (intranet.asianux.com [58.214.24.6]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by ozlabs.org (Postfix) with ESMTPS id 8684E2C0332 for ; Tue, 5 Mar 2013 12:58:51 +1100 (EST) Message-ID: <51355142.4070505@asianux.com> Date: Tue, 05 Mar 2013 09:58:26 +0800 From: Chen Gang MIME-Version: 1.0 To: Jiri Slaby Subject: Re: [PATCH] drivers/tty/hvc: using strlcpy instead of strncpy References: <512C2F5D.1080207@asianux.com> <512F343E.7010201@suse.cz> <512F3BC5.8020903@asianux.com> <512F3C60.9070409@asianux.com> <512F5FDC.3060000@suse.cz> In-Reply-To: <512F5FDC.3060000@suse.cz> Content-Type: text/plain; charset=UTF-8 Cc: Greg KH , tklauser@distanz.ch, wfp5p@virginia.edu, linuxppc-dev@lists.ozlabs.org, alan@linux.intel.com List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , 于 2013年02月28日 21:47, Jiri Slaby 写道: >>> when strlen(&pi->location_code[0]) == HVCS_CLC_LENGTH + 2 > It cannot, pi->location_code is defined as char[HVCS_CLC_LENGTH + 1]. > really, it is, I did not notice it. but I still prefer to modify it, but the patch should be changed such as: subject: beautify code: deleting useless judging code. comments: src buf len and dest buf len are the same, strcpy is better. contents: using strcpy instead of strncpy, and delete judging code. is it ok ? BTW: sorry for my reply is too late, and did not notify it, originally before. I have to do some urgent things, during these days. my father had a serious heart disease, and is in hospital. during these days, most of my time has to be in hospital. (God Bless, and thank Jesus Christ, my father is safe, now). within my company (Asianux), I also have something to do. excuse me: within this week, maybe can not get my mail reply, too. -- Chen Gang Asianux Corporation