From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965968AbdJQTbV (ORCPT ); Tue, 17 Oct 2017 15:31:21 -0400 Received: from esa7.dell-outbound.iphmx.com ([68.232.153.96]:3947 "EHLO esa7.dell-outbound.iphmx.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S965336AbdJQTbU (ORCPT ); Tue, 17 Oct 2017 15:31:20 -0400 From: X-LoopCount0: from 10.166.132.189 X-IronPort-AV: E=Sophos;i="5.43,392,1503378000"; d="scan'208";a="1166251451" X-DLP: DLP_GlobalPCIDSS To: CC: , , , , , , , , , , Subject: RE: [PATCH v9 03/17] platform/x86: dell-wmi: clean up wmi descriptor check Thread-Topic: [PATCH v9 03/17] platform/x86: dell-wmi: clean up wmi descriptor check Thread-Index: AQHTR3gEDNvniKfBj0GOs/N9kSG2p6LobZqg Date: Tue, 17 Oct 2017 19:31:13 +0000 Message-ID: <5da23d3566b04913aaef369cb7597e16@ausx13mpc120.AMER.DELL.COM> References: <20171017184445.c5endis7tzncplrz@pali> In-Reply-To: <20171017184445.c5endis7tzncplrz@pali> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-ms-exchange-transport-fromentityheader: Hosted x-originating-ip: [10.143.18.86] Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from base64 to 8bit by nfs id v9HJVReb013812 > -----Original Message----- > From: Pali Rohár [mailto:pali.rohar@gmail.com] > Sent: Tuesday, October 17, 2017 1:45 PM > To: Limonciello, Mario > Cc: dvhart@infradead.org; Andy Shevchenko ; > LKML ; platform-driver-x86@vger.kernel.org; Andy > Lutomirski ; quasisec@google.com; rjw@rjwysocki.net; > mjg59@google.com; hch@lst.de; Greg KH ; Alan Cox > > Subject: Re: [PATCH v9 03/17] platform/x86: dell-wmi: clean up wmi descriptor > check > > On Tuesday 17 October 2017 13:21:47 Mario Limonciello wrote: > > Some cases the wrong type was used for errors and checks can be > > done more cleanly. > > > > Signed-off-by: Mario Limonciello > > Reviewed-by: Edward O'Callaghan > > Suggested-by: Andy Shevchenko > > --- > > drivers/platform/x86/dell-wmi.c | 10 +++++----- > > 1 file changed, 5 insertions(+), 5 deletions(-) > > > > diff --git a/drivers/platform/x86/dell-wmi.c b/drivers/platform/x86/dell-wmi.c > > index 2cfaaa8faf0a..ece2fe341f01 100644 > > --- a/drivers/platform/x86/dell-wmi.c > > +++ b/drivers/platform/x86/dell-wmi.c > > @@ -663,19 +663,19 @@ static int dell_wmi_check_descriptor_buffer(struct > wmi_device *wdev) > > > > buffer = (u32 *)obj->buffer.pointer; > > > > - if (buffer[0] != 0x4C4C4544 && buffer[1] != 0x494D5720) { > > - dev_err(&wdev->dev, "Dell descriptor buffer has invalid signature > (%*ph)\n", > > - 8, buffer); > > + if (strncmp(obj->string.pointer, "DELL WMI", 8) != 0) { > > + dev_err(&wdev->dev, "Dell descriptor buffer has invalid signature > (%8ph)\n", > > + buffer); > > ret = -EINVAL; > > goto out; > > } > > > > if (buffer[2] != 0 && buffer[2] != 1) > > - dev_warn(&wdev->dev, "Dell descriptor buffer has unknown > version (%d)\n", > > + dev_warn(&wdev->dev, "Dell descriptor buffer has unknown > version (%u)\n", > > buffer[2]); > > To be correct, buffer[2] is of type "u32", not of type "unsigned". So > this patch does not fix it properly. > What's the proper solution then? Cast buffer[2] to a known type length like unsigned long and use %lu? From mboxrd@z Thu Jan 1 00:00:00 1970 From: Subject: RE: [PATCH v9 03/17] platform/x86: dell-wmi: clean up wmi descriptor check Date: Tue, 17 Oct 2017 19:31:13 +0000 Message-ID: <5da23d3566b04913aaef369cb7597e16@ausx13mpc120.AMER.DELL.COM> References: <20171017184445.c5endis7tzncplrz@pali> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from esa7.dell-outbound.iphmx.com ([68.232.153.96]:3947 "EHLO esa7.dell-outbound.iphmx.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S965336AbdJQTbU (ORCPT ); Tue, 17 Oct 2017 15:31:20 -0400 In-Reply-To: <20171017184445.c5endis7tzncplrz@pali> Content-Language: en-US Sender: platform-driver-x86-owner@vger.kernel.org List-ID: To: pali.rohar@gmail.com Cc: dvhart@infradead.org, andy.shevchenko@gmail.com, linux-kernel@vger.kernel.org, platform-driver-x86@vger.kernel.org, luto@kernel.org, quasisec@google.com, rjw@rjwysocki.net, mjg59@google.com, hch@lst.de, greg@kroah.com, gnomes@lxorguk.ukuu.org.uk PiAtLS0tLU9yaWdpbmFsIE1lc3NhZ2UtLS0tLQ0KPiBGcm9tOiBQYWxpIFJvaMOhciBbbWFpbHRv OnBhbGkucm9oYXJAZ21haWwuY29tXQ0KPiBTZW50OiBUdWVzZGF5LCBPY3RvYmVyIDE3LCAyMDE3 IDE6NDUgUE0NCj4gVG86IExpbW9uY2llbGxvLCBNYXJpbyA8TWFyaW9fTGltb25jaWVsbG9ARGVs bC5jb20+DQo+IENjOiBkdmhhcnRAaW5mcmFkZWFkLm9yZzsgQW5keSBTaGV2Y2hlbmtvIDxhbmR5 LnNoZXZjaGVua29AZ21haWwuY29tPjsNCj4gTEtNTCA8bGludXgta2VybmVsQHZnZXIua2VybmVs Lm9yZz47IHBsYXRmb3JtLWRyaXZlci14ODZAdmdlci5rZXJuZWwub3JnOyBBbmR5DQo+IEx1dG9t aXJza2kgPGx1dG9Aa2VybmVsLm9yZz47IHF1YXNpc2VjQGdvb2dsZS5jb207IHJqd0Byand5c29j a2kubmV0Ow0KPiBtamc1OUBnb29nbGUuY29tOyBoY2hAbHN0LmRlOyBHcmVnIEtIIDxncmVnQGty b2FoLmNvbT47IEFsYW4gQ294DQo+IDxnbm9tZXNAbHhvcmd1ay51a3V1Lm9yZy51az4NCj4gU3Vi amVjdDogUmU6IFtQQVRDSCB2OSAwMy8xN10gcGxhdGZvcm0veDg2OiBkZWxsLXdtaTogY2xlYW4g dXAgd21pIGRlc2NyaXB0b3INCj4gY2hlY2sNCj4gDQo+IE9uIFR1ZXNkYXkgMTcgT2N0b2JlciAy MDE3IDEzOjIxOjQ3IE1hcmlvIExpbW9uY2llbGxvIHdyb3RlOg0KPiA+IFNvbWUgY2FzZXMgdGhl IHdyb25nIHR5cGUgd2FzIHVzZWQgZm9yIGVycm9ycyBhbmQgY2hlY2tzIGNhbiBiZQ0KPiA+IGRv bmUgbW9yZSBjbGVhbmx5Lg0KPiA+DQo+ID4gU2lnbmVkLW9mZi1ieTogTWFyaW8gTGltb25jaWVs bG8gPG1hcmlvLmxpbW9uY2llbGxvQGRlbGwuY29tPg0KPiA+IFJldmlld2VkLWJ5OiBFZHdhcmQg TydDYWxsYWdoYW4gPHF1YXNpc2VjQGdvb2dsZS5jb20+DQo+ID4gU3VnZ2VzdGVkLWJ5OiBBbmR5 IFNoZXZjaGVua28gPGFuZHkuc2hldmNoZW5rb0BnbWFpbC5jb20+DQo+ID4gLS0tDQo+ID4gIGRy aXZlcnMvcGxhdGZvcm0veDg2L2RlbGwtd21pLmMgfCAxMCArKysrKy0tLS0tDQo+ID4gIDEgZmls ZSBjaGFuZ2VkLCA1IGluc2VydGlvbnMoKyksIDUgZGVsZXRpb25zKC0pDQo+ID4NCj4gPiBkaWZm IC0tZ2l0IGEvZHJpdmVycy9wbGF0Zm9ybS94ODYvZGVsbC13bWkuYyBiL2RyaXZlcnMvcGxhdGZv cm0veDg2L2RlbGwtd21pLmMNCj4gPiBpbmRleCAyY2ZhYWE4ZmFmMGEuLmVjZTJmZTM0MWYwMSAx MDA2NDQNCj4gPiAtLS0gYS9kcml2ZXJzL3BsYXRmb3JtL3g4Ni9kZWxsLXdtaS5jDQo+ID4gKysr IGIvZHJpdmVycy9wbGF0Zm9ybS94ODYvZGVsbC13bWkuYw0KPiA+IEBAIC02NjMsMTkgKzY2Mywx OSBAQCBzdGF0aWMgaW50IGRlbGxfd21pX2NoZWNrX2Rlc2NyaXB0b3JfYnVmZmVyKHN0cnVjdA0K PiB3bWlfZGV2aWNlICp3ZGV2KQ0KPiA+DQo+ID4gIAlidWZmZXIgPSAodTMyICopb2JqLT5idWZm ZXIucG9pbnRlcjsNCj4gPg0KPiA+IC0JaWYgKGJ1ZmZlclswXSAhPSAweDRDNEM0NTQ0ICYmIGJ1 ZmZlclsxXSAhPSAweDQ5NEQ1NzIwKSB7DQo+ID4gLQkJZGV2X2Vycigmd2Rldi0+ZGV2LCAiRGVs bCBkZXNjcmlwdG9yIGJ1ZmZlciBoYXMgaW52YWxpZCBzaWduYXR1cmUNCj4gKCUqcGgpXG4iLA0K PiA+IC0JCQk4LCBidWZmZXIpOw0KPiA+ICsJaWYgKHN0cm5jbXAob2JqLT5zdHJpbmcucG9pbnRl ciwgIkRFTEwgV01JIiwgOCkgIT0gMCkgew0KPiA+ICsJCWRldl9lcnIoJndkZXYtPmRldiwgIkRl bGwgZGVzY3JpcHRvciBidWZmZXIgaGFzIGludmFsaWQgc2lnbmF0dXJlDQo+ICglOHBoKVxuIiwN Cj4gPiArCQkJYnVmZmVyKTsNCj4gPiAgCQlyZXQgPSAtRUlOVkFMOw0KPiA+ICAJCWdvdG8gb3V0 Ow0KPiA+ICAJfQ0KPiA+DQo+ID4gIAlpZiAoYnVmZmVyWzJdICE9IDAgJiYgYnVmZmVyWzJdICE9 IDEpDQo+ID4gLQkJZGV2X3dhcm4oJndkZXYtPmRldiwgIkRlbGwgZGVzY3JpcHRvciBidWZmZXIg aGFzIHVua25vd24NCj4gdmVyc2lvbiAoJWQpXG4iLA0KPiA+ICsJCWRldl93YXJuKCZ3ZGV2LT5k ZXYsICJEZWxsIGRlc2NyaXB0b3IgYnVmZmVyIGhhcyB1bmtub3duDQo+IHZlcnNpb24gKCV1KVxu IiwNCj4gPiAgCQkJYnVmZmVyWzJdKTsNCj4gDQo+IFRvIGJlIGNvcnJlY3QsIGJ1ZmZlclsyXSBp cyBvZiB0eXBlICJ1MzIiLCBub3Qgb2YgdHlwZSAidW5zaWduZWQiLiBTbw0KPiB0aGlzIHBhdGNo IGRvZXMgbm90IGZpeCBpdCBwcm9wZXJseS4NCj4gDQoNCldoYXQncyB0aGUgcHJvcGVyIHNvbHV0 aW9uIHRoZW4/ICANCkNhc3QgYnVmZmVyWzJdIHRvIGEga25vd24gdHlwZSBsZW5ndGggbGlrZSB1 bnNpZ25lZCBsb25nIGFuZCB1c2UgJWx1Pw0K