linux-kernel - Re: [PATCH V3 1/1] Drivers: hv: Implement the file copy service

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] [day] [month] [year] [list]

Message-ID: <20140121191326.GA16171@aepfle.de>
Date:	Tue, 21 Jan 2014 20:13:26 +0100
From:	Olaf Hering <olaf@...fle.de>
To:	"K. Y. Srinivasan" <kys@...rosoft.com>
Cc:	gregkh@...uxfoundation.org, linux-kernel@...r.kernel.org,
	devel@...uxdriverproject.org, apw@...onical.com,
	jasowang@...hat.com
Subject: Re: [PATCH V3 1/1] Drivers: hv: Implement the file copy service

On Tue, Jan 21, K. Y. Srinivasan wrote:

Looks better. A few comments below:


> +++ b/drivers/hv/hv_fcopy.c

> +static bool daemon_died;

This is a write only variable.

> +static u32 daemon_version;

This is a write only variable.

> +static int fcopy_handle_handshake(u32 version)
> +{
> +	pr_info("FCP: user-mode registering done. Daemon version: %d\n",
> +		version);
> +	fcopy_transaction.active = false;
> +	daemon_version = version;

I think fcopy_handle_handshake should proactivly reject requests if
version is not 0.

There is an unused FCOPY_VERSION as well. I think this should not be in
a header. Instead the kernel should handle all known versions and reject
everything it does not know about.

> +	syslog(LOG_INFO, "Target file name: %s\n", target_fname);

Newline in syslog

> +				if (mkdir((char *)smsg->path_name, 0755)) {
> +					syslog(LOG_ERR, "Failed to create %s\n",

Newline in syslog

> +		syslog(LOG_INFO, "File: %s exists\n", target_fname);

Newline in syslog

> +		syslog(LOG_INFO, "Open Failed: %s\n", strerror(errno));

Newline in syslog

> +				syslog(LOG_ERR, "Read error: %s\n",

Newline in syslog

> +			syslog(LOG_ERR, "Unknown operation: %d\n",

Newline in syslog

Olaf
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/