Hi, these patches add ipv6 support to SpiceUri, so it can be used in SPICE_PROXY. Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1335239 v4 contains changes suggested by Christophe, Victor, Frediano: - Added check for scheme - Tests check for error messages - Port 0 is considered invalid - Fixed memory leaks v3 per Frediano's review: - Added test for port out of range (0, 65535) - reset all values of SpiceUri before parsing v2 contains changes suggested by Frediano and Victor: - more tests (missing port, missing ending bracket) - tests are defined in array, so it is easier to add new test cases - follow ipv6 address format http://user:password@[ipv6]:port Thanks, Pavel Grunt (8): tests: Add test for SpiceUri spice-uri: Mark parameter as unused spice-uri: Add missing include spice-uri: Reset SpiceUri before parsing spice-uri: Do not allow empty port string spice-uri: Check if port is in allowed range spice-uri: Validate uri scheme spice-uri: Add ipv6 support src/spice-uri.c | 79 ++++++++++++++++++++------- tests/Makefile.am | 2 + tests/test-spice-uri.c | 142 +++++++++++++++++++++++++++++++++++++++++++++++++ 3 files changed, 203 insertions(+), 20 deletions(-) create mode 100644 tests/test-spice-uri.c -- 2.8.3 _______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel