Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/2194297/?format=api
{ "id": 2194297, "url": "http://patchwork.ozlabs.org/api/patches/2194297/?format=api", "web_url": "http://patchwork.ozlabs.org/project/qemu-devel/patch/20260208103958.121455-4-heifetz@yandex-team.com/", "project": { "id": 14, "url": "http://patchwork.ozlabs.org/api/projects/14/?format=api", "name": "QEMU Development", "link_name": "qemu-devel", "list_id": "qemu-devel.nongnu.org", "list_email": "qemu-devel@nongnu.org", "web_url": "", "scm_url": "", "webscm_url": "", "list_archive_url": "", "list_archive_url_format": "", "commit_url_format": "" }, "msgid": "<20260208103958.121455-4-heifetz@yandex-team.com>", "list_archive_url": null, "date": "2026-02-08T10:39:57", "name": "[v3,3/4] system/physmem.c: remove useless assertion of block", "commit_ref": null, "pull_url": null, "state": "new", "archived": false, "hash": "caddbfcde47e7e9779291527e57466f5bbf06004", "submitter": { "id": 92578, "url": "http://patchwork.ozlabs.org/api/people/92578/?format=api", "name": "Sergei Heifetz", "email": "heifetz@yandex-team.com" }, "delegate": null, "mbox": "http://patchwork.ozlabs.org/project/qemu-devel/patch/20260208103958.121455-4-heifetz@yandex-team.com/mbox/", "series": [ { "id": 491417, "url": "http://patchwork.ozlabs.org/api/series/491417/?format=api", "web_url": "http://patchwork.ozlabs.org/project/qemu-devel/list/?series=491417", "date": "2026-02-08T10:39:54", "name": "Fix a few minor issues reported by Coverity", "version": 3, "mbox": "http://patchwork.ozlabs.org/series/491417/mbox/" } ], "comments": "http://patchwork.ozlabs.org/api/patches/2194297/comments/", "check": "pending", "checks": "http://patchwork.ozlabs.org/api/patches/2194297/checks/", "tags": {}, "related": [], "headers": { "Return-Path": "<qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>", "X-Original-To": "incoming@patchwork.ozlabs.org", "Delivered-To": "patchwork-incoming@legolas.ozlabs.org", "Authentication-Results": [ "legolas.ozlabs.org;\n\tdkim=pass (1024-bit key;\n unprotected) header.d=yandex-team.com header.i=@yandex-team.com\n header.a=rsa-sha256 header.s=default header.b=i7jLCQZk;\n\tdkim-atps=neutral", "legolas.ozlabs.org;\n spf=pass (sender SPF authorized) smtp.mailfrom=nongnu.org\n (client-ip=209.51.188.17; helo=lists.gnu.org;\n envelope-from=qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org;\n receiver=patchwork.ozlabs.org)", "mail-nwsmtp-smtp-corp-main-68.klg.yp-c.yandex.net;\n dkim=pass header.i=@yandex-team.com" ], "Received": [ "from lists.gnu.org (lists.gnu.org [209.51.188.17])\n\t(using TLSv1.2 with cipher ECDHE-ECDSA-AES256-GCM-SHA384 (256/256 bits))\n\t(No client certificate requested)\n\tby legolas.ozlabs.org (Postfix) with ESMTPS id 4f84Bh5DXZz1xtr\n\tfor <incoming@patchwork.ozlabs.org>; Sun, 08 Feb 2026 21:41:48 +1100 (AEDT)", "from localhost ([::1] helo=lists1p.gnu.org)\n\tby lists.gnu.org with esmtp (Exim 4.90_1)\n\t(envelope-from <qemu-devel-bounces@nongnu.org>)\n\tid 1vp2EJ-0004FE-Nv; Sun, 08 Feb 2026 05:41:27 -0500", "from eggs.gnu.org ([2001:470:142:3::10])\n by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256)\n (Exim 4.90_1) (envelope-from <heifetz@yandex-team.com>)\n id 1vp2EI-0004Dn-BV; Sun, 08 Feb 2026 05:41:26 -0500", "from forwardcorp1d.mail.yandex.net ([178.154.239.200])\n by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256)\n (Exim 4.90_1) (envelope-from <heifetz@yandex-team.com>)\n id 1vp2EG-00079z-Pm; Sun, 08 Feb 2026 05:41:26 -0500", "from mail-nwsmtp-smtp-corp-main-68.klg.yp-c.yandex.net\n (mail-nwsmtp-smtp-corp-main-68.klg.yp-c.yandex.net\n [IPv6:2a02:6b8:c42:94a9:0:640:a3fa:0])\n by forwardcorp1d.mail.yandex.net (Yandex) with ESMTPS id 33F9B80605;\n Sun, 08 Feb 2026 13:41:23 +0300 (MSK)", "from heifetz-lin.yandex-team.ru (unknown [2a02:6bf:8080:d25::1:35])\n by mail-nwsmtp-smtp-corp-main-68.klg.yp-c.yandex.net\n (smtpcorp/Yandex) with ESMTPSA id WeNYFo1Apa60-Q2qTpPXd;\n Sun, 08 Feb 2026 13:41:22 +0300" ], "X-Yandex-Fwd": "1", "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed; d=yandex-team.com;\n s=default; t=1770547283;\n bh=Y8T3sQ7myYRXFcyLO/RPNCMgHh3wQJni+Ls0PXgNJQQ=;\n h=Message-Id:Date:In-Reply-To:Cc:Subject:References:To:From;\n b=i7jLCQZkWzBe8aGVU3aKtmvAWaW4IFPZQN0u9K8kSDtk0pWDpRkIV0Ei0ZMXq7xGe\n 7iI1jMcd4GH2HRx+c6qxZf97ZbMxqfbW4z5ctbqaph8g/TB6Gu0Q8fvoVZy0mAUJT0\n 5aCL/BhuSqtQsx5vM9N2QL4mBtEAX6rCFhufX2TE=", "From": "Sergei Heifetz <heifetz@yandex-team.com>", "To": "qemu-devel@nongnu.org", "Cc": "qemu-trivial@nongnu.org", "Subject": "[PATCH v3 3/4] system/physmem.c: remove useless assertion of block", "Date": "Sun, 8 Feb 2026 15:39:57 +0500", "Message-Id": "<20260208103958.121455-4-heifetz@yandex-team.com>", "X-Mailer": "git-send-email 2.34.1", "In-Reply-To": "<20260208103958.121455-1-heifetz@yandex-team.com>", "References": "<20260208103958.121455-1-heifetz@yandex-team.com>", "MIME-Version": "1.0", "Content-Transfer-Encoding": "8bit", "Received-SPF": "pass client-ip=178.154.239.200;\n envelope-from=heifetz@yandex-team.com; helo=forwardcorp1d.mail.yandex.net", "X-Spam_score_int": "-20", "X-Spam_score": "-2.1", "X-Spam_bar": "--", "X-Spam_report": "(-2.1 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1,\n DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1,\n RCVD_IN_VALIDITY_RPBL_BLOCKED=0.001, RCVD_IN_VALIDITY_SAFE_BLOCKED=0.001,\n SPF_HELO_NONE=0.001, SPF_PASS=-0.001 autolearn=ham autolearn_force=no", "X-Spam_action": "no action", "X-BeenThere": "qemu-devel@nongnu.org", "X-Mailman-Version": "2.1.29", "Precedence": "list", "List-Id": "qemu development <qemu-devel.nongnu.org>", "List-Unsubscribe": "<https://lists.nongnu.org/mailman/options/qemu-devel>,\n <mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>", "List-Archive": "<https://lists.nongnu.org/archive/html/qemu-devel>", "List-Post": "<mailto:qemu-devel@nongnu.org>", "List-Help": "<mailto:qemu-devel-request@nongnu.org?subject=help>", "List-Subscribe": "<https://lists.nongnu.org/mailman/listinfo/qemu-devel>,\n <mailto:qemu-devel-request@nongnu.org?subject=subscribe>", "Errors-To": "qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org", "Sender": "qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org" }, "content": "It is useless to assert that block is not NULL because\nit is already dereferenced in the first line of the function.\n\nWe could split the declaration and initialization of oldsize,\nbut then we would need to remove the const qualifier.\nThis seems worse, as the assertion would be almost useless anyway.\n\nSigned-off-by: Sergei Heifetz <heifetz@yandex-team.com>\n---\n system/physmem.c | 2 --\n 1 file changed, 2 deletions(-)", "diff": "diff --git a/system/physmem.c b/system/physmem.c\nindex b0311f4531..317b359ebe 100644\n--- a/system/physmem.c\n+++ b/system/physmem.c\n@@ -2057,8 +2057,6 @@ int qemu_ram_resize(RAMBlock *block, ram_addr_t newsize, Error **errp)\n const ram_addr_t oldsize = block->used_length;\n const ram_addr_t unaligned_size = newsize;\n \n- assert(block);\n-\n newsize = TARGET_PAGE_ALIGN(newsize);\n newsize = REAL_HOST_PAGE_ALIGN(newsize);\n \n", "prefixes": [ "v3", "3/4" ] }