On Sun, Nov 26, 2023 at 10:54:38PM -0600, Steve French wrote:
> on this one - I lean toward leaving it in (although technically
> unused) since may reduce future errors by being clear that this is not
> an error case and may be a bit clearer to read to some. No strong
> opinion though on this.
>
I will undo this change and update the Coverity triage data instead.
> On Sun, Nov 26, 2023 at 10:52 PM Pierre Mariani
> <pierre.mariani@xxxxxxxxx> wrote:
> >
> > rc does not need to be set to any value in this location as it gets set to other
> > values is all subsequent logical branches before being used.
> > Fixes Coverity 1562035 Unused value.
> >
> > Signed-off-by: Pierre Mariani <pierre.mariani@xxxxxxxxx>
> > ---
> > fs/smb/client/connect.c | 1 -
> > 1 file changed, 1 deletion(-)
> >
> > diff --git a/fs/smb/client/connect.c b/fs/smb/client/connect.c
> > index f896f60c924b..449d56802692 100644
> > --- a/fs/smb/client/connect.c
> > +++ b/fs/smb/client/connect.c
> > @@ -1770,7 +1770,6 @@ cifs_get_tcp_session(struct smb3_fs_context *ctx,
> > tcp_ses, (struct sockaddr *)&ctx->dstaddr);
> > if (tcp_ses->smbd_conn) {
> > cifs_dbg(VFS, "RDMA transport established\n");
> > - rc = 0;
> > goto smbd_connected;
> > } else {
> > rc = -ENOENT;
> > --
> > 2.39.2
> >
>
>
> --
> Thanks,
>
> Steve
