From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-dl1-f42.google.com (mail-dl1-f42.google.com [74.125.82.42]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 3337041C6A for ; Mon, 4 May 2026 23:15:57 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=74.125.82.42 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1777936558; cv=none; b=KRRKueRoxlG63snooQ62ciIiDOw1SYVn35sLCJItJNVxYaqQa68nluhy+t6KOXxJCWj9tUbl9c4qSc3v2jvOLlzKbTGWWI8Tx2t/oPSAFcoyCPYhp4DDnhDNCMcV1NLIYNS+ymHJYnWA8bmlw61TVwrcyKofW3sfUoHr5m2VYQs= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1777936558; c=relaxed/simple; bh=6kA5f8/Eq1uu4hnjHTbRSXA1fMelDXAzD0GtxiZTMBo=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=k+aYNkQzDTYTFBJhfPW+x1Ey1FBeFd62bNxcRQ7YMoB4dDSa1zhsTfNsWo9tYgudn3L7C+MGydpgdFhhvCuHsDWCfptvsRWa5BjV3SNLtFkk0B/7S84yiJm3SWP+qS6Y1MQGWJb2ES3ee0IsXzGTUgKIYT+EaSr+44jsXEXNYos= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=OHUVOf2U; arc=none smtp.client-ip=74.125.82.42 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="OHUVOf2U" Received: by mail-dl1-f42.google.com with SMTP id a92af1059eb24-12c45281a06so7198549c88.1 for ; Mon, 04 May 2026 16:15:57 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1777936556; x=1778541356; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=uJUaPNEnLT6eI3Sz+OplXWx4CX0ycNqOaqliP4b8tDU=; b=OHUVOf2UM8g4vLb7uYSU6h5tYF5iG+pxFI2BhfjLj060+C7P68bEIfcaaEvlK4ZBSQ LL1JLXxMo6dQcxGpR4kfrMjmIzZeBOn3MXoq/Fp5YtHTMf6g3tzhSAOrOiYLccFDRE7X mwfOJKzvHMJOZXDFHM9nawoPZrwr24xEoxusT3zqkmJVTRqJjkLCT0srQB4zDVAf6uHU fB9TR/qUpJwAVKnZr+6vsaMMBXVqIZb1pNhURGF+nm86hFAd6xEhHPmD+rcbOAc0V492 IU6esuFqC3h0BauoAMFynEWDIXlJ7kU4GeFaNGRkjJDY50+2ccRUgZGXRrOvuygPlKbd 1plg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1777936556; x=1778541356; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-gg:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=uJUaPNEnLT6eI3Sz+OplXWx4CX0ycNqOaqliP4b8tDU=; b=aloKtuU8ojt52BjbtAusYl9FFlYD+aO6PgcDnGjcvX54j0wP4ZoIAiQfHUw7caSDJS TmXVl1YBlVaHrmcgF38KkDW12JZ8eecMnOJY6frePfH6zqQ7zQxiTnn2zdN7gC3TJeB9 KKscfId0Di2tJQ+DrmhAlnn4yYjl8STQLYUP7Flb1bxBnUJskdhtW3Gaef1tuUhReswy oGGxqJtesaoLMyQlCC+4AIvpn2/sKsHR8nofJhqP/Tr8jWl0fAao/8/5FIJd6lYBR8Kh MXXywQPrhYdgkolE7+3UWG4J13WVHLRq/y8+fBDx+U0XYIOc8nbdu77tSo7obpH3JImy lUEw== X-Forwarded-Encrypted: i=1; AFNElJ9HvrYc1Lq0cSivHmV9hR1q6hNFSWxuRVmZ4kl+4cX2u0SlRe2YANwBnM5/thLqtDOboCxwUVzBc8DFRC0=@vger.kernel.org X-Gm-Message-State: AOJu0YxfqrjzjZY3q2KUh5P4USie+24efyxYM8UxVY+jtSuRhZFQ9Gqa PokRJE+/093saK/WnKvFnbc2OKnSTi/4raMnLPcPRNFKbI7q2vEe0ATL X-Gm-Gg: AeBDiesBBhKhy0vJNSrLq+KZTsYKayjWoI/+Lnb2d+jGOiEEqLyulqmLI2KukW2KTYc X+/hTePPnIMl0clmMlrNq1XM3nHG5VAL7dOX3dbd0c3m5ihDHgT0tu9U4gbXklzJi1lbo7J/YaG AkBDiTKVfWa+zDNwx3foKSYy4zRsIi2djVz1uZztR3UqC8HMRluQRjkpz6oK9js2/B+5xh2hzqA 7A0XTnE1Q1typ85d+NspHNajKDAjR4B5vJHq4pJaWomY2HbGqnelRuYKKIey+ATBl/OIDVfuNvE ZGtjBfXqG7evd2RgvVHDD1c2I9T75gdiYdNsvKXDG1qZShvbSkeXnMHis3Mo7a+eDXrQQAyStrt Vp3QZLLSikUUi838NsOomnjsX/RdbTb+DtLhJT8A+7aqa2ZZ1VXuyBwNv+hsx3nsQetV49l3Ocw RAETWLHfhl7xdFtfM8fWIcls8ZtGcnXNg20j8tknTRq3m2pbUOVC0F5eq+8sckKlD2QZKEipww4 7maVmVApVjT2Q== X-Received: by 2002:a05:7022:605:b0:12c:8b9:7208 with SMTP id a92af1059eb24-130b1b67874mr477748c88.26.1777936556135; Mon, 04 May 2026 16:15:56 -0700 (PDT) Received: from google.com ([2a00:79e0:2ebe:8:5b87:9b19:32e2:2981]) by smtp.gmail.com with ESMTPSA id a92af1059eb24-12df84252c0sm20605766c88.10.2026.05.04.16.15.54 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 04 May 2026 16:15:55 -0700 (PDT) Date: Mon, 4 May 2026 16:15:51 -0700 From: Dmitry Torokhov To: Benjamin Tissoires Cc: Jiri Kosina , Filipe =?utf-8?B?TGHDrW5z?= , Bastien Nocera , Ping Cheng , Jason Gerecke , Viresh Kumar , Johan Hovold , Alex Elder , Greg Kroah-Hartman , Lee Jones , Icenowy Zheng , linux-input@vger.kernel.org, linux-kernel@vger.kernel.org, greybus-dev@lists.linaro.org, linux-staging@lists.linux.dev, linux-usb@vger.kernel.org Subject: Re: [PATCH v3 4/4] HID: wacom: use __free(kfree) to clean up temporary buffers Message-ID: References: <20260504-wip-fix-core-v3-0-ce1f11f4968f@kernel.org> <20260504-wip-fix-core-v3-4-ce1f11f4968f@kernel.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20260504-wip-fix-core-v3-4-ce1f11f4968f@kernel.org> Hi Benjamin, On Mon, May 04, 2026 at 10:47:25AM +0200, Benjamin Tissoires wrote: > @@ -386,10 +381,11 @@ static void wacom_feature_mapping(struct hid_device *hdev, > case WACOM_HID_WD_OFFSETRIGHT: > case WACOM_HID_WD_OFFSETBOTTOM: > /* read manually */ > - n = hid_report_len(field->report); > - data = hid_alloc_report_buf(field->report, GFP_KERNEL); > + u8 *data __free(kfree) = hid_alloc_report_buf(field->report, GFP_KERNEL); > + > if (!data) > break; > + n = hid_report_len(field->report); > data[0] = field->report->id; > ret = wacom_get_report(hdev, HID_FEATURE_REPORT, > data, n, WAC_CMD_RETRIES); > @@ -400,7 +396,6 @@ static void wacom_feature_mapping(struct hid_device *hdev, > hid_warn(hdev, "%s: could not retrieve sensor offsets\n", > __func__); > } > - kfree(data); > break; > } I'd recommend establishing a new scope for the "data", otherwise it is fragile. If there was another label below then this cleanup would explode since current scope of "data" is from the declaration point until the end of the switch statement. Having a dedicated scope makes lifertime explicit. Thanks. -- Dmitry