linux-kernel - Re: [PATCH 4.19 188/191] cifs: log warning message (once) if out of disk space

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20200222125931.GC14067@amd>
Date:   Sat, 22 Feb 2020 13:59:31 +0100
From:   Pavel Machek <pavel@...x.de>
To:     Greg Kroah-Hartman <gregkh@...uxfoundation.org>
Cc:     linux-kernel@...r.kernel.org, stable@...r.kernel.org,
        Steve French <stfrench@...rosoft.com>,
        Oleg Kravtsov <oleg@...era.com>,
        Ronnie Sahlberg <lsahlber@...hat.com>,
        Pavel Shilovsky <pshilov@...rosoft.com>,
        Sasha Levin <sashal@...nel.org>
Subject: Re: [PATCH 4.19 188/191] cifs: log warning message (once) if out of
 disk space

On Fri 2020-02-21 08:42:41, Greg Kroah-Hartman wrote:
> From: Steve French <stfrench@...rosoft.com>
> 
> [ Upstream commit d6fd41905ec577851734623fb905b1763801f5ef ]
> 
> We ran into a confusing problem where an application wasn't checking
> return code on close and so user didn't realize that the application
> ran out of disk space.  log a warning message (once) in these
> cases. For example:
> 
>   [ 8407.391909] Out of space writing to \\oleg-server\small-share

Out of space can happen on any filesystem, and yes, it can be
confusing. But why is cifs so special that we warn here (and not
elsewhere) and why was this marked for stable?

Best regards,
								Pavel

> +++ b/fs/cifs/smb2pdu.c
> @@ -3425,6 +3425,9 @@ smb2_writev_callback(struct mid_q_entry *mid)
>  				     wdata->cfile->fid.persistent_fid,
>  				     tcon->tid, tcon->ses->Suid, wdata->offset,
>  				     wdata->bytes, wdata->result);
> +		if (wdata->result == -ENOSPC)
> +			printk_once(KERN_WARNING "Out of space writing to %s\n",
> +				    tcon->treeName);
>  	} else
>  		trace_smb3_write_done(0 /* no xid */,
>  				      wdata->cfile->fid.persistent_fid,

-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

Download attachment "signature.asc" of type "application/pgp-signature" (182 bytes)