From 7af3bcb4bc6a4408f91b193de4b0eadc11ccf70b Mon Sep 17 00:00:00 2001
From: Daniel Kesselberg <mail@danielkesselberg.de>
Date: Thu, 16 Jan 2020 14:04:23 +0100
Subject: [PATCH 1/2] Add test to trigger "Trying to access array offset on
 value of type int"

Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
---
 .../lib/AppFramework/OCS/BaseResponseTest.php | 62 +++++++++++++++++++
 1 file changed, 62 insertions(+)
 create mode 100644 tests/lib/AppFramework/OCS/BaseResponseTest.php

diff --git a/tests/lib/AppFramework/OCS/BaseResponseTest.php b/tests/lib/AppFramework/OCS/BaseResponseTest.php
new file mode 100644
index 0000000000..8a86ae13e7
--- /dev/null
+++ b/tests/lib/AppFramework/OCS/BaseResponseTest.php
@@ -0,0 +1,62 @@
+<?php
+
+declare(strict_types=1);
+
+/**
+ * @copyright 2020 Daniel Kesselberg <mail@danielkesselberg.de>
+ *
+ * @author 2020 Daniel Kesselberg <mail@danielkesselberg.de>
+ *
+ * @license GNU AGPL version 3 or any later version
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU Affero General Public License as
+ * published by the Free Software Foundation, either version 3 of the
+ * License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU Affero General Public License for more details.
+ *
+ * You should have received a copy of the GNU Affero General Public License
+ * along with this program. If not, see <http://www.gnu.org/licenses/>.
+ *
+ */
+
+namespace Test\AppFramework\Middleware;
+
+
+use OC\AppFramework\OCS\BaseResponse;
+
+class BaseResponseTest extends \Test\TestCase {
+
+	public function testToXml(): void {
+
+		/** @var BaseResponse $response */
+		$response = $this->createMock(BaseResponse::class);
+
+		$writer = new \XMLWriter();
+		$writer->openMemory();
+		$writer->setIndent(false);
+		$writer->startDocument();
+
+		$data = [
+			'hello' => 'hello',
+			'information' => [
+				'@test' => 'some data',
+				'someElement' => 'withAttribute',
+			],
+			'value without key',
+		];
+
+		$this->invokePrivate($response, 'toXml', [$data, $writer]);
+		$writer->endDocument();
+
+		$this->assertEquals(
+			"<?xml version=\"1.0\"?>\n<hello>hello</hello><information test=\"some data\"><someElement>withAttribute</someElement></information><element>value without key</element>\n",
+			$writer->outputMemory(true)
+		);
+	}
+
+}

From 96d1640a373475a02a3e2c261b73a80e46a57e94 Mon Sep 17 00:00:00 2001
From: Daniel Kesselberg <mail@danielkesselberg.de>
Date: Thu, 16 Jan 2020 14:09:25 +0100
Subject: [PATCH 2/2] Use strpos to check that @ is the first char

Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
---
 lib/private/AppFramework/OCS/BaseResponse.php | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/private/AppFramework/OCS/BaseResponse.php b/lib/private/AppFramework/OCS/BaseResponse.php
index 60a4fb57ab..68456289be 100644
--- a/lib/private/AppFramework/OCS/BaseResponse.php
+++ b/lib/private/AppFramework/OCS/BaseResponse.php
@@ -126,7 +126,7 @@ abstract class BaseResponse extends Response   {
 	 */
 	protected function toXML(array $array, \XMLWriter $writer) {
 		foreach ($array as $k => $v) {
-			if ($k[0] === '@') {
+			if (\is_string($k) && strpos($k, '@') === 0) {
 				$writer->writeAttribute(substr($k, 1), $v);
 				continue;
 			}