{"id":816124,"url":"http://patchwork.ozlabs.org/api/covers/816124/?format=json","web_url":"http://patchwork.ozlabs.org/project/ubuntu-kernel/cover/20170920102707.8266-1-colin.king@canonical.com/","project":{"id":15,"url":"http://patchwork.ozlabs.org/api/projects/15/?format=json","name":"Ubuntu Kernel","link_name":"ubuntu-kernel","list_id":"kernel-team.lists.ubuntu.com","list_email":"kernel-team@lists.ubuntu.com","web_url":null,"scm_url":null,"webscm_url":null,"list_archive_url":"","list_archive_url_format":"","commit_url_format":""},"msgid":"<20170920102707.8266-1-colin.king@canonical.com>","list_archive_url":null,"date":"2017-09-20T10:27:04","name":"[0/3] UBUNTU: SAUCE: opennsl: fix static analysis issues","submitter":{"id":2900,"url":"http://patchwork.ozlabs.org/api/people/2900/?format=json","name":"Colin Ian King","email":"colin.king@canonical.com"},"mbox":"http://patchwork.ozlabs.org/project/ubuntu-kernel/cover/20170920102707.8266-1-colin.king@canonical.com/mbox/","series":[{"id":4075,"url":"http://patchwork.ozlabs.org/api/series/4075/?format=json","web_url":"http://patchwork.ozlabs.org/project/ubuntu-kernel/list/?series=4075","date":"2017-09-20T10:27:04","name":"UBUNTU: SAUCE: opennsl: fix static analysis issues","version":1,"mbox":"http://patchwork.ozlabs.org/series/4075/mbox/"}],"comments":"http://patchwork.ozlabs.org/api/covers/816124/comments/","headers":{"Return-Path":"","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":"ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.ubuntu.com\n\t(client-ip=91.189.94.19; helo=huckleberry.canonical.com;\n\tenvelope-from=kernel-team-bounces@lists.ubuntu.com;\n\treceiver=)","Received":["from huckleberry.canonical.com (huckleberry.canonical.com\n\t[91.189.94.19])\n\tby ozlabs.org (Postfix) with ESMTP id 3xxwsZ5Dntz9s7B;\n\tWed, 20 Sep 2017 20:27:14 +1000 (AEST)","from localhost ([127.0.0.1] helo=huckleberry.canonical.com)\n\tby huckleberry.canonical.com with esmtp (Exim 4.86_2)\n\t(envelope-from )\n\tid 1ducDk-0002HM-UY; Wed, 20 Sep 2017 10:27:08 +0000","from youngberry.canonical.com ([91.189.89.112])\n\tby huckleberry.canonical.com with esmtps\n\t(TLS1.0:DHE_RSA_AES_128_CBC_SHA1:128)\n\t(Exim 4.86_2) (envelope-from )\n\tid 1ducDj-0002Fs-Vh\n\tfor kernel-team@lists.ubuntu.com; Wed, 20 Sep 2017 10:27:07 +0000","from 1.general.cking.uk.vpn ([10.172.193.212] helo=localhost)\n\tby youngberry.canonical.com with esmtpsa\n\t(TLS1.0:RSA_AES_256_CBC_SHA1:32)\n\t(Exim 4.76) (envelope-from )\n\tid 1ducDj-00080e-JN; Wed, 20 Sep 2017 10:27:07 +0000"],"From":"Colin King ","To":"kernel-team@lists.ubuntu.com","Subject":"[PATCH 0/3] UBUNTU: SAUCE: opennsl: fix static analysis issues","Date":"Wed, 20 Sep 2017 11:27:04 +0100","Message-Id":"<20170920102707.8266-1-colin.king@canonical.com>","X-Mailer":"git-send-email 2.14.1","X-BeenThere":"kernel-team@lists.ubuntu.com","X-Mailman-Version":"2.1.20","Precedence":"list","List-Id":"Kernel team discussions ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","MIME-Version":"1.0","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"kernel-team-bounces@lists.ubuntu.com","Sender":"\"kernel-team\" "},"content":"From: Colin Ian King \n\nStatic analysis with CoverityScan on the opennsl driver has picked up 2\nsets of bugs and some redundant code. The following 3 patches address\nthese issues to make the driver static analysis clean.\n\nBugLink: https://launchpad.net/bugs/1718388\n\nColin Ian King (3):\n opennsl: bcm-knet: check for null sinfo to avoid a null pointer\n dereference\n opennsl: bcm-knet: remove redundant null checks on dev->name\n opennsl: bde: check for out-of-bounds index io.dev\n\n .../systems/bde/linux/user/kernel/linux-user-bde.c | 42 ++++++++++++++++++++++\n .../linux/kernel/modules/bcm-knet/bcm-knet.c | 7 ++--\n 2 files changed, 46 insertions(+), 3 deletions(-)"}