Re: [PATCH 2/2] cifs: check returned value for error

From: Muhammad Usama Anjum
Date: Wed Oct 05 2022 - 01:17:50 EST


On 10/5/22 12:01 AM, Paulo Alcantara wrote:
Muhammad Usama Anjum <usama.anjum@xxxxxxxxxxxxx> writes:

smb311_decode_neg_context() can return error. Its return value should be
checked for errors.

Signed-off-by: Muhammad Usama Anjum <usama.anjum@xxxxxxxxxxxxx>
---
fs/cifs/smb2pdu.c | 7 +++++--
1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/fs/cifs/smb2pdu.c b/fs/cifs/smb2pdu.c
index 2bf43c892ae6..c6e37352dbe1 100644

This patch doesn't apply in Steve's for-next branch[1]. If it still
makes sense, please rebase and resend.

[1] git://git.samba.org/sfrench/cifs-2.6.git
I've just applied on the latest next next-20221004 tag. This patch doesn't apply. It seems some other patch has merged and this patch isn't needed.

Can you apply only the first patch "[PATCH 1/2] cifs: remove initialization value" from this series? Or should I send that only as v2?