OCA\Files_Antivirus\Scanner\Socket

Complexity

Total Complexity

3

Size/Duplication

Total Lines	27
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

0%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	initScanner()	0	20	2

<?php
/**
 * Copyright (c) 2017 Viktar Dubiniuk <dubiniuk@owncloud.com>
 * This file is licensed under the Affero General Public License version 3 or
 * later.
 * See the COPYING-README file.
 */


namespace OCA\Files_Antivirus\Scanner;

class Socket extends Daemon {

	public function __construct($config){
		$this->appConfig = $config;
	}

	public function initScanner(){
		parent::initScanner();

		$this->writeHandle = stream_socket_client(
			'unix://' . $this->appConfig->getAvSocket(), $errorCode, $errorMessage, 5
		);
		if (!$this->getWriteHandle()) {
			throw new InitException(
				sprintf(
					'Could not connect to socket "%s": %s (code %d)',
					$this->appConfig->getAvSocket(),
					$errorMessage,
					$errorCode
				)
			);
		}

		// request scan from the daemon
		@fwrite($this->getWriteHandle(), "nINSTREAM\n");

It seems like you do not handle an error condition here. This can introduce security issues, and is generally not recommended.

	}
}