Sanitize host before calling pm (#49644)

packages/flutter_tools/lib/src/base/net.dart

@@ -233,3 +233,13 @@ class _MemoryIOSink implements IOSink {
   @override
   Future<void> flush() async { }
 }
+
+/// Returns [true] if [address] is an IPv6 address.
+bool isIPv6Address(String address) {
+  try {
+    Uri.parseIPv6Address(address);
+    return true;
+  } on FormatException {
+    return false;
+  }
+}

packages/flutter_tools/lib/src/fuchsia/fuchsia_device.dart

@@ -13,6 +13,7 @@ import '../base/context.dart';
 import '../base/file_system.dart';
 import '../base/io.dart';
 import '../base/logger.dart';
+import '../base/net.dart';
 import '../base/os.dart';
 import '../base/process.dart';
 import '../base/time.dart';
@@ -512,14 +513,10 @@ class FuchsiaDevice extends Device {
   @override
   void clearLogs() {}
 
-  bool get ipv6 {
-    try {
-      Uri.parseIPv6Address(id);
-      return true;
-    } on FormatException {
-      return false;
-    }
-  }
+  bool _ipv6;
+
+  /// [true] if the current host address is IPv6.
+  bool get ipv6 => _ipv6 ??= isIPv6Address(id);
 
   /// List the ports currently running a dart observatory.
   Future<List<int>> servicePorts() async {

packages/flutter_tools/lib/src/fuchsia/fuchsia_pm.dart

@@ -5,6 +5,7 @@
 import '../base/common.dart';
 import '../base/file_system.dart';
 import '../base/io.dart';
+import '../base/net.dart';
 import '../base/process.dart';
 import '../convert.dart';
 import '../globals.dart' as globals;
@@ -109,6 +110,9 @@ class FuchsiaPM {
     if (fuchsiaArtifacts.pm == null) {
       throwToolExit('Fuchsia pm tool not found');
     }
+    if (isIPv6Address(host.split('%').first)) {
+      host = '[${host.replaceAll('%', '%25')}]';
+    }
     final List<String> command = <String>[
       fuchsiaArtifacts.pm.path,
       'serve',

packages/flutter_tools/test/general.shard/fuchsia/fuchsia_pm_test.dart

+// Copyright 2014 The Flutter Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+import 'dart:async';
+
+import 'package:flutter_tools/src/base/context.dart';
+import 'package:flutter_tools/src/base/file_system.dart';
+import 'package:flutter_tools/src/base/io.dart';
+import 'package:flutter_tools/src/base/process.dart';
+import 'package:flutter_tools/src/fuchsia/fuchsia_pm.dart';
+import 'package:flutter_tools/src/fuchsia/fuchsia_sdk.dart';
+import 'package:mockito/mockito.dart';
+
+import '../../src/common.dart';
+import '../../src/context.dart';
+import '../../src/mocks.dart';
+
+void main() {
+  group('FuchsiaPM', () {
+    MockFile pm;
+    MockProcessManager mockProcessManager;
+    MockFuchsiaArtifacts mockFuchsiaArtifacts;
+
+    setUp(() {
+      pm = MockFile();
+      when(pm.path).thenReturn('pm');
+
+      mockFuchsiaArtifacts = MockFuchsiaArtifacts();
+      when(mockFuchsiaArtifacts.pm).thenReturn(pm);
+
+      mockProcessManager = MockProcessManager();
+    });
+
+    testUsingContext('serve - IPv4 address', () async {
+      when(mockProcessManager.start(any)).thenAnswer((_) {
+        return Future<Process>.value(createMockProcess());
+      });
+
+      await FuchsiaPM().serve('<repo>', '127.0.0.1', 43819);
+
+      verify(mockProcessManager.start(<String>[
+        'pm',
+        'serve',
+        '-repo',
+        '<repo>',
+        '-l',
+        '127.0.0.1:43819',
+      ])).called(1);
+    }, overrides: <Type, Generator>{
+      FuchsiaArtifacts: () => mockFuchsiaArtifacts,
+      ProcessManager: () => mockProcessManager,
+    });
+
+    testUsingContext('serve - IPv6 address', () async {
+      when(mockProcessManager.start(any)).thenAnswer((_) {
+        return Future<Process>.value(createMockProcess());
+      });
+
+      await FuchsiaPM().serve('<repo>', 'fe80::ec4:7aff:fecc:ea8f%eno2', 43819);
+
+      verify(mockProcessManager.start(<String>[
+        'pm',
+        'serve',
+        '-repo',
+        '<repo>',
+        '-l',
+        '[fe80::ec4:7aff:fecc:ea8f%25eno2]:43819',
+      ])).called(1);
+    }, overrides: <Type, Generator>{
+      FuchsiaArtifacts: () => mockFuchsiaArtifacts,
+      ProcessManager: () => mockProcessManager,
+    });
+  });
+}
+
+class MockFuchsiaArtifacts extends Mock implements FuchsiaArtifacts {}
+class MockProcessUtils extends Mock implements ProcessUtils {}
+class MockFile extends Mock implements File {}
+class MockProcess extends Mock implements Process {}
+class MockProcessManager extends Mock implements ProcessManager {}