Jon,
Another bogus check, it must be removed.
Apply the attached patch.
Jean-Louis
On 09/03/17 02:35 PM, Jon LaBadie wrote:
> On Thu, Mar 09, 2017 at 09:10:49AM -0500, Jean-Louis Martineau wrote:
>> Jon,
>>
>> You can remove the faullty 4 lines.
>> 3.4 can do multiple backup in parallel.
>>
>> Instead of using amdump_client/amdumpd, you should use ambackup/ambackupd.
>> amdump_client/amdumpd is the old protocol and is kept only for
>> compatibility with 3.3
>>
>> Jean-Louis
> Just tried ambackup/ambackupd. On line 144 of ambackupd:
>
> if (defined $req->{'options'}->{'hostname'} and \
> $req->{'options'}->{'hostname'} ne $hostname) {
>
> The right side expression is comparing the server name
> and the client name. As they are not equal ambackupd
> fails with the message:
>
> ERROR hostname mismatch mums.jgcomp.com : cyber.jgcomp.com
>
> I haven't explored where "req" gets set and whether its
> hostname member should be server or client.
>
> Jon
This message is the property of CARBONITE, INC. and may contain confidential or
privileged information.
If this message has been delivered to you by mistake, then do not copy or
deliver this message to anyone. Instead, destroy it and notify me by reply
e-mail.
diff --git a/server-src/ambackupd.pl b/server-src/ambackupd.pl
index 662fd2e..e0d2b66 100644
--- a/server-src/ambackupd.pl
+++ b/server-src/ambackupd.pl
@@ -141,9 +141,6 @@ sub setup_streams {
my $rep = '';
my @result_messages;
my $hostname = $peer;
- if (defined $req->{'options'}->{'hostname'} and $req->{'options'}->{'hostname'} ne $hostname) {
- push @$errors, "hostname mismatch $req->{'options'}->{'hostname'} : $hostname";
- } else {
my $cmd_line = $req->{'lines'}[1];
if (!$cmd_line) {
push @$errors, "No command specified in REQ packet";
@@ -278,7 +275,6 @@ sub setup_streams {
} else {
push @$errors, "Invalid command '$cmd' specified in REQ packet";
}
- }
}
if (@result_messages) {
if (@$errors || @{$req->{'errors'}}) {
