From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jani Nikula Subject: Re: [PATCH] format-security: move static strings to const Date: Thu, 06 Apr 2017 11:48:48 +0300 Message-ID: <87mvbtzztb.fsf@intel.com> References: <20170405214711.GA5711@beast> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Cc: linux-mips@linux-mips.org, linux-decnet-user@lists.sourceforge.net, kernel@stlinux.com, Qianqian Xie , Mugunthan V N , Tony Lindgren , Viresh Kumar , Rasmus Villemoes , dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, Eric Dumazet , "Maciej W. Rozycki" , Christian Gromm , Daniel Vetter , Ingo Molnar , Andrey Shvetsov , devel@driverdev.osuosl.org, Kejian Yan , Felipe Balbi , Russell King , Jarod Wilson , linux-hippi@sunsite.dk, Daode Huang , linux-serial@vger.kernel.org, Jiri Slaby , Andrew Morton Return-path: In-Reply-To: <20170405214711.GA5711@beast> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" List-Id: netdev.vger.kernel.org T24gVGh1LCAwNiBBcHIgMjAxNywgS2VlcyBDb29rIDxrZWVzY29va0BjaHJvbWl1bS5vcmc+IHdy b3RlOgo+IFdoaWxlIGV4YW1pbmluZyBvdXRwdXQgZnJvbSB0cmlhbCBidWlsZHMgd2l0aCAtV2Zv cm1hdC1zZWN1cml0eSBlbmFibGVkLAo+IG1hbnkgc3RyaW5ncyB3ZXJlIGZvdW5kIHRoYXQgc2hv dWxkIGJlIGRlZmluZWQgYXMgImNvbnN0Iiwgb3IgYXMgYSBjaGFyCj4gYXJyYXkgaW5zdGVhZCBv ZiBjaGFyIHBvaW50ZXIuIFRoaXMgbWFrZXMgc29tZSBzdGF0aWMgYW5hbHlzaXMgZWFzaWVyLAo+ IGJ5IHByb2R1Y2luZyBmZXdlciBmYWxzZSBwb3NpdGl2ZXMuCj4KPiBBcyB0aGVzZSBhcmUgYWxs IHRyaXZpYWwgY2hhbmdlcywgaXQgc2VlbWVkIGJlc3QgdG8gcHV0IHRoZW0gYWxsIGluCj4gYSBz aW5nbGUgcGF0Y2ggcmF0aGVyIHRoYW4gY2hvcHBpbmcgdGhlbSB1cCBwZXIgbWFpbnRhaW5lci4K Cj4gZGlmZiAtLWdpdCBhL2RyaXZlcnMvZ3B1L2RybS9kcm1fZmJfaGVscGVyLmMgYi9kcml2ZXJz L2dwdS9kcm0vZHJtX2ZiX2hlbHBlci5jCj4gaW5kZXggZjZkNGQ5NzAwNzM0Li4xZmY5ZDU5MTJi ODMgMTAwNjQ0Cj4gLS0tIGEvZHJpdmVycy9ncHUvZHJtL2RybV9mYl9oZWxwZXIuYwo+ICsrKyBi L2RyaXZlcnMvZ3B1L2RybS9kcm1fZmJfaGVscGVyLmMKPiBAQCAtMjMzMSw3ICsyMzMxLDcgQEAg RVhQT1JUX1NZTUJPTChkcm1fZmJfaGVscGVyX2hvdHBsdWdfZXZlbnQpOwo+ICBpbnQgX19pbml0 IGRybV9mYl9oZWxwZXJfbW9kaW5pdCh2b2lkKQo+ICB7Cj4gICNpZiBkZWZpbmVkKENPTkZJR19G UkFNRUJVRkZFUl9DT05TT0xFX01PRFVMRSkgJiYgIWRlZmluZWQoQ09ORklHX0VYUEVSVCkKPiAt CWNvbnN0IGNoYXIgKm5hbWUgPSAiZmJjb24iOwo+ICsJY29uc3QgY2hhciBuYW1lW10gPSAiZmJj b24iOwoKSSdkIGFsd2F5cyB3cml0ZSB0aGUgZm9ybWVyIG91dCBvZiBoYWJpdC4gV2h5IHNob3Vs ZCBJIHN0YXJ0IHVzaW5nIHRoZQpsYXR0ZXI/IFdoYXQgbWFrZXMgaXQgYmV0dGVyPwoKV2hhdCBr ZWVwcyB0aGUga2VybmVsIGZyb20gYWNjdW11bGF0aW5nIHRvbnMgbW9yZSBvZiB0aGUgZm9ybWVy PwoKSGVyZSdzIGFuIGludGVyZXN0aW5nIGNvbXBhcmlzb24gb2YgdGhlIGdlbmVyYXRlZCBjb2Rl LiBJJ20gYSBiaXQKc3VycHJpc2VkIGJ5IHdoYXQgZ2NjIGRvZXMsIEkgd291bGQgaGF2ZSBleHBl Y3RlZCBubyBkaWZmZXJlbmNlLCBsaWtlCmNsYW5nLiBodHRwczovL2dvZGJvbHQub3JnL2cvT2Rx VXZOCgpUaGUgb3RoZXIgY2hhbmdlcyBhZGRpbmcgY29uc3QgaW4gdGhpcyBwYXRjaCBhcmUsIG9m IGNvdXJzZSwgZ29vZC4KCgpCUiwKSmFuaS4KCi0tIApKYW5pIE5pa3VsYSwgSW50ZWwgT3BlbiBT b3VyY2UgVGVjaG5vbG9neSBDZW50ZXIKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX18KZHJpLWRldmVsIG1haWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMuZnJl ZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlzdHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGlu Zm8vZHJpLWRldmVsCg== From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jani Nikula Subject: Re: [PATCH] format-security: move static strings to const Date: Thu, 06 Apr 2017 11:48:48 +0300 Message-ID: <87mvbtzztb.fsf@intel.com> References: <20170405214711.GA5711@beast> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <20170405214711.GA5711@beast> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Kees Cook , Andrew Morton Cc: linux-mips@linux-mips.org, linux-decnet-user@lists.sourceforge.net, kernel@stlinux.com, Qianqian Xie , Mugunthan V N , Tony Lindgren , Viresh Kumar , Rasmus Villemoes , dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, Eric Dumazet , "Maciej W. Rozycki" , Christian Gromm , Daniel Vetter , Ingo Molnar , Andrey Shvetsov , devel@driverdev.osuosl.org, Kejian Yan , Felipe Balbi , Russell King , Jarod Wilson , linux-hippi@sunsite.dk, Daode Huang , linux-serial@vger.kernel.org, Jiri Slaby List-Id: linux-pm@vger.kernel.org T24gVGh1LCAwNiBBcHIgMjAxNywgS2VlcyBDb29rIDxrZWVzY29va0BjaHJvbWl1bS5vcmc+IHdy b3RlOgo+IFdoaWxlIGV4YW1pbmluZyBvdXRwdXQgZnJvbSB0cmlhbCBidWlsZHMgd2l0aCAtV2Zv cm1hdC1zZWN1cml0eSBlbmFibGVkLAo+IG1hbnkgc3RyaW5ncyB3ZXJlIGZvdW5kIHRoYXQgc2hv dWxkIGJlIGRlZmluZWQgYXMgImNvbnN0Iiwgb3IgYXMgYSBjaGFyCj4gYXJyYXkgaW5zdGVhZCBv ZiBjaGFyIHBvaW50ZXIuIFRoaXMgbWFrZXMgc29tZSBzdGF0aWMgYW5hbHlzaXMgZWFzaWVyLAo+ IGJ5IHByb2R1Y2luZyBmZXdlciBmYWxzZSBwb3NpdGl2ZXMuCj4KPiBBcyB0aGVzZSBhcmUgYWxs IHRyaXZpYWwgY2hhbmdlcywgaXQgc2VlbWVkIGJlc3QgdG8gcHV0IHRoZW0gYWxsIGluCj4gYSBz aW5nbGUgcGF0Y2ggcmF0aGVyIHRoYW4gY2hvcHBpbmcgdGhlbSB1cCBwZXIgbWFpbnRhaW5lci4K Cj4gZGlmZiAtLWdpdCBhL2RyaXZlcnMvZ3B1L2RybS9kcm1fZmJfaGVscGVyLmMgYi9kcml2ZXJz L2dwdS9kcm0vZHJtX2ZiX2hlbHBlci5jCj4gaW5kZXggZjZkNGQ5NzAwNzM0Li4xZmY5ZDU5MTJi ODMgMTAwNjQ0Cj4gLS0tIGEvZHJpdmVycy9ncHUvZHJtL2RybV9mYl9oZWxwZXIuYwo+ICsrKyBi L2RyaXZlcnMvZ3B1L2RybS9kcm1fZmJfaGVscGVyLmMKPiBAQCAtMjMzMSw3ICsyMzMxLDcgQEAg RVhQT1JUX1NZTUJPTChkcm1fZmJfaGVscGVyX2hvdHBsdWdfZXZlbnQpOwo+ICBpbnQgX19pbml0 IGRybV9mYl9oZWxwZXJfbW9kaW5pdCh2b2lkKQo+ICB7Cj4gICNpZiBkZWZpbmVkKENPTkZJR19G UkFNRUJVRkZFUl9DT05TT0xFX01PRFVMRSkgJiYgIWRlZmluZWQoQ09ORklHX0VYUEVSVCkKPiAt CWNvbnN0IGNoYXIgKm5hbWUgPSAiZmJjb24iOwo+ICsJY29uc3QgY2hhciBuYW1lW10gPSAiZmJj b24iOwoKSSdkIGFsd2F5cyB3cml0ZSB0aGUgZm9ybWVyIG91dCBvZiBoYWJpdC4gV2h5IHNob3Vs ZCBJIHN0YXJ0IHVzaW5nIHRoZQpsYXR0ZXI/IFdoYXQgbWFrZXMgaXQgYmV0dGVyPwoKV2hhdCBr ZWVwcyB0aGUga2VybmVsIGZyb20gYWNjdW11bGF0aW5nIHRvbnMgbW9yZSBvZiB0aGUgZm9ybWVy PwoKSGVyZSdzIGFuIGludGVyZXN0aW5nIGNvbXBhcmlzb24gb2YgdGhlIGdlbmVyYXRlZCBjb2Rl LiBJJ20gYSBiaXQKc3VycHJpc2VkIGJ5IHdoYXQgZ2NjIGRvZXMsIEkgd291bGQgaGF2ZSBleHBl Y3RlZCBubyBkaWZmZXJlbmNlLCBsaWtlCmNsYW5nLiBodHRwczovL2dvZGJvbHQub3JnL2cvT2Rx VXZOCgpUaGUgb3RoZXIgY2hhbmdlcyBhZGRpbmcgY29uc3QgaW4gdGhpcyBwYXRjaCBhcmUsIG9m IGNvdXJzZSwgZ29vZC4KCgpCUiwKSmFuaS4KCi0tIApKYW5pIE5pa3VsYSwgSW50ZWwgT3BlbiBT b3VyY2UgVGVjaG5vbG9neSBDZW50ZXIKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX18KZHJpLWRldmVsIG1haWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMuZnJl ZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlzdHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGlu Zm8vZHJpLWRldmVsCg== From mboxrd@z Thu Jan 1 00:00:00 1970 Received: with ECARTIS (v1.0.0; list linux-mips); Thu, 06 Apr 2017 10:49:12 +0200 (CEST) Received: from mga06.intel.com ([134.134.136.31]:32360 "EHLO mga06.intel.com" rhost-flags-OK-OK-OK-OK) by eddie.linux-mips.org with ESMTP id S23990519AbdDFItEL3bHK (ORCPT ); Thu, 6 Apr 2017 10:49:04 +0200 Received: from fmsmga003.fm.intel.com ([10.253.24.29]) by orsmga104.jf.intel.com with ESMTP; 06 Apr 2017 01:49:01 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.37,283,1488873600"; d="scan'208";a="842574461" Received: from jnikula-mobl.fi.intel.com (HELO localhost) ([10.237.72.162]) by FMSMGA003.fm.intel.com with ESMTP; 06 Apr 2017 01:48:49 -0700 From: Jani Nikula To: Kees Cook , Andrew Morton Cc: Tony Lindgren , Russell King , "Maciej W. Rozycki" , Ralf Baechle , Arnd Bergmann , Greg Kroah-Hartman , "Rafael J. Wysocki" , Viresh Kumar , Daniel Vetter , Sean Paul , David Airlie , Yisen Zhuang , Salil Mehta , Thomas Bogendoerfer , Jes Sorensen , Jiri Slaby , Patrice Chotard , "David S. Miller" , James Hogan , Paul Burton , Matt Redfearn , Paolo Bonzini , Ingo Molnar , Rasmus Villemoes , Mugunthan V N , Felipe Balbi , Jarod Wilson , Florian Westphal , Antonio Quartulli , Dmitry Torokhov , Kejian Yan , Daode Huang , Qianqian Xie , Philippe Reynes , Colin Ian King , Eric Dumazet , Christian Gromm , Andrey Shvetsov , Jason Litzinger , WANG Cong , linux-arm-kernel@lists.infradead.org, linux-omap@vger.kernel.org, linux-mips@linux-mips.org, linux-pm@vger.kernel.org, dri-devel@lists.freedesktop.org, netdev@vger.kernel.org, linux-hippi@sunsite.dk, devel@driverdev.osuosl.org, kernel@stlinux.com, linux-serial@vger.kernel.org, linux-decnet-user@lists.sourceforge.net, linux-kernel@vger.kernel.org Subject: Re: [PATCH] format-security: move static strings to const In-Reply-To: <20170405214711.GA5711@beast> Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo References: <20170405214711.GA5711@beast> Date: Thu, 06 Apr 2017 11:48:48 +0300 Message-ID: <87mvbtzztb.fsf@intel.com> MIME-Version: 1.0 Content-Type: text/plain Return-Path: X-Envelope-To: <"|/home/ecartis/ecartis -s linux-mips"> (uid 0) X-Orcpt: rfc822;linux-mips@linux-mips.org Original-Recipient: rfc822;linux-mips@linux-mips.org X-archive-position: 57578 X-ecartis-version: Ecartis v1.0.0 Sender: linux-mips-bounce@linux-mips.org Errors-to: linux-mips-bounce@linux-mips.org X-original-sender: jani.nikula@linux.intel.com Precedence: bulk List-help: List-unsubscribe: List-software: Ecartis version 1.0.0 List-Id: linux-mips X-List-ID: linux-mips List-subscribe: List-owner: List-post: List-archive: X-list: linux-mips On Thu, 06 Apr 2017, Kees Cook wrote: > While examining output from trial builds with -Wformat-security enabled, > many strings were found that should be defined as "const", or as a char > array instead of char pointer. This makes some static analysis easier, > by producing fewer false positives. > > As these are all trivial changes, it seemed best to put them all in > a single patch rather than chopping them up per maintainer. > diff --git a/drivers/gpu/drm/drm_fb_helper.c b/drivers/gpu/drm/drm_fb_helper.c > index f6d4d9700734..1ff9d5912b83 100644 > --- a/drivers/gpu/drm/drm_fb_helper.c > +++ b/drivers/gpu/drm/drm_fb_helper.c > @@ -2331,7 +2331,7 @@ EXPORT_SYMBOL(drm_fb_helper_hotplug_event); > int __init drm_fb_helper_modinit(void) > { > #if defined(CONFIG_FRAMEBUFFER_CONSOLE_MODULE) && !defined(CONFIG_EXPERT) > - const char *name = "fbcon"; > + const char name[] = "fbcon"; I'd always write the former out of habit. Why should I start using the latter? What makes it better? What keeps the kernel from accumulating tons more of the former? Here's an interesting comparison of the generated code. I'm a bit surprised by what gcc does, I would have expected no difference, like clang. https://godbolt.org/g/OdqUvN The other changes adding const in this patch are, of course, good. BR, Jani. -- Jani Nikula, Intel Open Source Technology Center