USE_CXXSTD=	c++98

	${INSTALL_DATA} ${FILESDIR}/FREEBSDUSERS \

		${STAGEDIR}${DOCSDIR}/FREEBSDUSERS

post-install:

	@${ECHO_MSG} "********************************************************"

	@${ECHO_MSG} "*           IMPORTANT           IMPORTANT              *"

	@${ECHO_MSG} "********************************************************"

	@${ECHO_MSG} "*  Please read ${DOCSDIR}/FREEBSDUSERS"

	@${ECHO_MSG} "*  for valuable information regarding some of the"

	@${ECHO_MSG} "*  problems users have reported running ${PORTNAME} on"

	@${ECHO_MSG} "*  FreeBSD, and the solutions to many, if not all"

	@${ECHO_MSG} "*  of them."

	@${ECHO_MSG} "*"

	@${ECHO_MSG} "********************************************************"

First off a HUGE thanks go out to fjwcash@

for his relentless, and tireless work with this. I can't thank you enough!

That said, and for all others whom claim it doesn't work;

The answer(s) are as follows:

The following makes things work:

On the receiving server:

    sysctl net.inet6.ip6.v6only=0

    install bbcp-20150116.txz

On the sending server:

    sysctl net.inet6.ip6.v6only=0

    install bbcp-20150116.txz

    add "-4 c" to the start of the bbcp command

With the above, I am able to use the latest version of the bbcp package

to send data between servers.

Command-line used:

bbcp -4 c -N io "zfs send -I ${localpool}/${localfs}/${site}@${lastsnap} \

     ${localpool}/${localfs}/${site}@${yesterday}" \

     ${user}@${remotehost}:"${zfs} recv ${recvopts} ${rempool}"

Taken from the bbcp website (when in doubt, google it out):

--ipv4 [who] | -4 [who]

uses the IPv4 TCP stack for command processing and source-target connections.

This option is incompatible with IPv6 addresses. The optional who argument

restricts IPv4 mode to the command if c is specified, the source if s is

specified or the target if t is specified. The default is cst (i.e. everywhere).

If who is omitted and --ipv4 is the last option on the command line it must

be followed by the -- option (double dash). 

So, it looks like the "-h" output should be updated to use:

[-4 [cst]]

Or similar, with an explanation of what c, s, and t stand for?

  -c ( Compression )

  -s ( Streams )

  -t ( tlim - (timelimit) )

On another note, "-4 s" and "-4 t" end with segmentation faults. 

The only option that works is "-4 c".

So, until, or unless I'm able to overcome the problems with the -s, and

-t switches; don't use them.

Thanks again, fjwcash@ !

--Chris

--- src/bbcp_Args.C.orig	2015-01-14 19:57:32 UTC

+++ src/bbcp_Args.C

@@ -58,7 +58,7 @@ char *operator%(char *optarg)

          bbcp_Opt *p = this;

          do if (i <= p->Optmaxl && i >= p->Optminl &&

                !strncmp(p->Optword, optarg, i)) return p->Optvalu;

-            while(p = p->Optnext);

+            while((p = p->Optnext));

          return 0;

         }

@@ -203,7 +203,7 @@ char bbcp_Args::getopt()

    if (optspec[1] == '.')

       {if (argval && *argval == '-')

           if (inStream) arg_stream.RetToken();

-             else Aloc--;

+             }else{ Aloc--;

        argval = 0;

        return *optspec;

       }

--- src/bbcp_BuffPool.C.orig	2015-01-14 20:12:20 UTC

+++ src/bbcp_BuffPool.C

@@ -32,7 +32,7 @@

 #include <inttypes.h>

 #include <sys/mman.h>

-#if defined(MACOS) || defined(AIX)

+#if defined(MACOS) || defined(AIX) || defined(FREEBSD)

 #define memalign(pgsz,amt) valloc(amt)

 #else

 #include <malloc.h>

@@ -84,14 +84,14 @@ bbcp_BuffPool::~bbcp_BuffPool()

 // Free all of the buffers in the empty queue

 //

-   while(currp = last_empty)

+   while((currp = last_empty))

         {last_empty = last_empty->next; delete currp;}

 //cerr <<bbcp_Debug.Who <<"Bdestroy num " <<j++ <<" @ " <<hex <<(int)currp <<dec <<endl;

 // Free all full buffers

 //

    FullPool.Lock();

-   while(currp = next_full)

+   while((currp = next_full))

         {next_full = next_full->next; delete currp;}

    FullPool.UnLock();

 }

--- src/bbcp_Config.C.orig	2015-01-14 19:58:17 UTC

+++ src/bbcp_Config.C

@@ -265,7 +265,7 @@ void bbcp_Config::Arguments(int argc, char **argv, int

 // Process the options

 //

-   while (c=arglist.getopt())

+   while ((c=arglist.getopt()))

      { switch(c)

        {

        case 'a': Options |= bbcp_APPEND | bbcp_ORDER;

@@ -775,8 +775,8 @@ int bbcp_Config::ConfigInit(int argc, char **argv)

 // Use the config file, if present

 //

-   if (ConfigFN = getenv("bbcp_CONFIGFN"))

-      {if (retc = Configure(ConfigFN)) return retc;}

+   if ((ConfigFN = getenv("bbcp_CONFIGFN")))

+      {if ((retc = Configure(ConfigFN))) return retc;}

       else {

       // Use configuration file in the home directory, if any

       //

@@ -786,7 +786,7 @@ int bbcp_Config::ConfigInit(int argc, char **argv)

       strcpy(ConfigFN, homedir); strcat(ConfigFN, cfn);

       if (stat(ConfigFN, &buf))

          {retc = 0; free(ConfigFN); ConfigFN = 0;}

-         else if (retc = Configure(ConfigFN)) return retc;

+         else if ((retc = Configure(ConfigFN))) return retc;

      }

 // Establish the FD limit

@@ -1361,7 +1361,7 @@ int bbcp_Config::HostAndPort(const char *what, char *p

     if (*hn == ':' && hn++ && *hn)

        {errno = 0;

         pnum = strtol(hn, (char **)NULL, 10);

-        if (!pnum && errno || pnum > 65535)

+        if ((!pnum && errno) || pnum > 65535)

            {bbcp_Fmsg("Config",what,"port invalid -", hn); return -1;}

        }

--- src/bbcp_File.C.orig	2015-01-14 20:12:57 UTC

+++ src/bbcp_File.C

@@ -44,7 +44,7 @@

 #ifdef FREEBSD

 #undef ENODATA

-#define ENODATA ENOATTRR

+#define ENODATA ENOATTR

 #endif

 /******************************************************************************/

@@ -153,7 +153,7 @@ bbcp_Buffer *bbcp_File::getBuffer(long long offset)

 // Find a buffer

 //

-   if (bp = nextbuff)

+   if ((bp = nextbuff))

       while(bp && bp->boff != offset) {pp = bp; bp = bp->next;}

 // If we found a buffer, unchain it

--- src/bbcp_FileSpec.C.orig	2015-01-14 20:00:12 UTC

+++ src/bbcp_FileSpec.C

@@ -174,7 +174,7 @@ int bbcp_FileSpec::Compose(long long did, char *dpath,

 // Get the current state of the file or directory

 //

-        if (retc = FSp->Stat(targpath, &Targ)) targetsz = 0;

+        if ((retc = FSp->Stat(targpath, &Targ))) targetsz = 0;

    else if (Targ.Otype == 'p' && (bbcp_Config.Options & bbcp_XPIPE))

                                {targetsz =  0; return 0;}

    else if (Targ.Otype != 'f') {targetsz = -1; return 0;}

@@ -211,7 +211,7 @@ int bbcp_FileSpec::Create_Link()

 // in the directory. This will later be set to the true mode if it differs.

 //

    DEBUG("Make link " <<targpath <<" -> " <<Info.SLink);

-   if (retc = FSp->MKLnk(Info.SLink, targpath))

+   if ((retc = FSp->MKLnk(Info.SLink, targpath)))

       return bbcp_Emsg("Create_Link", retc, "creating link", targpath);

 // All done

@@ -232,7 +232,7 @@ int bbcp_FileSpec::Create_Path()

 // in the directory. This will later be set to the true mode if it differs.

 //

    DEBUG("Make path " <<Info.mode <<' ' <<targpath);

-   if (retc = FSp->MKDir(targpath, bbcp_Config.ModeDC))

+   if ((retc = FSp->MKDir(targpath, bbcp_Config.ModeDC)))

       if (retc == -EEXIST) return 0;

          else return bbcp_Emsg("Create_Path", retc, "creating path", targpath);

@@ -764,7 +764,7 @@ void bbcp_FileSpec::BuildPaths()

                         else bbcp_Config.srcPath = PS_New;

              PS_Prv = PS_New; PS_Cur = PS_New->next;

             }

-         if (*cp = delim) cp++;

+         if ((*cp = delim)) cp++;

         }

 }

--- src/bbcp_NetAddr.C.orig	2015-01-14 20:13:30 UTC

+++ src/bbcp_NetAddr.C

@@ -49,7 +49,7 @@

 #ifndef s6_addr32

 #if   defined(SUN)

 #define s6_addr32 _S6_un._S6_u32

-#elif defined(MACOS)

+#elif defined(MACOS) || defined(FREEBSD)

 #define s6_addr32 __u6_addr.__u6_addr32

 #endif

 #endif

--- src/bbcp_NetAddrInfo.C.orig	2015-01-14 20:14:07 UTC

+++ src/bbcp_NetAddrInfo.C

@@ -67,7 +67,7 @@

 #ifndef s6_addr32

 #if   defined(SUN)

 #define s6_addr32 _S6_un._S6_u32

-#elif defined(MACOS)

+#elif defined(MACOS) || defined(FREEBSD)

 #define s6_addr32 __u6_addr.__u6_addr32

 #endif

 #endif

--- src/bbcp_Node.C.orig	2015-01-14 20:02:51 UTC

+++ src/bbcp_Node.C

@@ -94,7 +94,7 @@ void *bbcp_Connect(void *protp)

      bbcp_Link     *link;

      int            retc;

-     if (link = bbcp_Net.Connect(bbcp_Config.CBhost, bbcp_Config.CBport))

+     if ((link = bbcp_Net.Connect(bbcp_Config.CBhost, bbcp_Config.CBport)))

         {if ((retc = protocol->Login(link, 0)) < 0)

             {delete link; link = 0;}

         }

@@ -354,8 +354,8 @@ int bbcp_Node::RecvFile(bbcp_FileSpec *fp, bbcp_Node *

             oflag = O_WRONLY | O_CREAT | O_TRUNC;

            }

    else if (bbcp_Config.Options & bbcp_APPEND)

-           {if (retc = fp->WriteSigFile()) return retc;

-            if (startoff = fp->targetsz) oflag = O_WRONLY;

+           {if ((retc = fp->WriteSigFile())) return retc;

+            if ((startoff = fp->targetsz)) oflag = O_WRONLY;

                else oflag = O_CREAT | O_WRONLY;

            }

    else    oflag = O_WRONLY | O_CREAT | O_EXCL;

@@ -368,7 +368,7 @@ int bbcp_Node::RecvFile(bbcp_FileSpec *fp, bbcp_Node *

 // Tell the user what we are bout to do

 //

-   if (bbcp_Config.Options & bbcp_BLAB | bbcp_Config.Progint)

+   if ((bbcp_Config.Options & bbcp_BLAB) | bbcp_Config.Progint)

       if (bbcp_Config.Options & bbcp_APPEND)

          {char buff[32];

           sprintf(buff, "%lld", startoff);

@@ -464,7 +464,7 @@ int bbcp_Node::RecvFile(bbcp_FileSpec *fp, bbcp_Node *

 // Wait for the expansion thread to end

 //

    if (bbcp_Config.Options & bbcp_COMPRESS)

-      {if (tretc = (long)bbcp_Thread_Wait(cxp->TID)) retc = 128;

+      {if ((tretc = (long)bbcp_Thread_Wait(cxp->TID))) retc = 128;

        DEBUG("File expansion ended; rc=" <<tretc);

       }

@@ -478,7 +478,7 @@ int bbcp_Node::RecvFile(bbcp_FileSpec *fp, bbcp_Node *

 // Make sure each thread has terminated normally

 //

    for (i = 0; i < dlcount; i++)

-       {if (tretc = (long)bbcp_Thread_Wait(link_tid[i])) retc = 128;

+       {if ((tretc = (long)bbcp_Thread_Wait(link_tid[i]))) retc = 128;

         DEBUG("Thread " <<link_tid[i] <<" stream " <<i <<" ended; rc=" <<tretc);

        }

@@ -618,7 +618,7 @@ int bbcp_Node::SendFile(bbcp_FileSpec *fp)

 // Wait for compression thread to end

 //

    if (bbcp_Config.Options & bbcp_COMPRESS)

-      {if (tretc = (long)bbcp_Thread_Wait(cxp->TID)) retc = 128;

+      {if ((tretc = (long)bbcp_Thread_Wait(cxp->TID))) retc = 128;

        DEBUG("File compression ended; rc=" <<tretc);

        delete cxp;

       }

@@ -626,7 +626,7 @@ int bbcp_Node::SendFile(bbcp_FileSpec *fp)

 // Make sure each link thread has terminated normally.

 //

    for (i = 0; i < iocount; i++)

-       {if (tretc = (long)bbcp_Thread_Wait(link_tid[i])) retc = 128;

+       {if ((tretc = (long)bbcp_Thread_Wait(link_tid[i]))) retc = 128;

         DEBUG("Thread " <<link_tid[i] <<" stream " <<i <<" ended; rc=" <<tretc);

        }

@@ -735,7 +735,7 @@ int bbcp_Node::Outgoing(bbcp_Protocol *protocol)

 // Establish the control connection first

 //

-   if (link = bbcp_Net.Connect(bbcp_Config.CBhost, bbcp_Config.CBport, 3))

+   if ((link = bbcp_Net.Connect(bbcp_Config.CBhost, bbcp_Config.CBport, 3)))

       if ((retc = protocol->Login(link, 0)) < 0)

          {delete link; link = 0;}

--- src/bbcp_ProcMon.C.orig	2015-01-14 20:04:50 UTC

+++ src/bbcp_ProcMon.C

@@ -132,7 +132,7 @@ void bbcp_ProcMon::Start(pid_t monit, bbcp_Node *Remot

 // Run a thread to start the monitor

 //

-   if (retc = bbcp_Thread_Run(bbcp_MonProc, (void *)this, &mytid))

+   if ((retc = bbcp_Thread_Run(bbcp_MonProc, (void *)this, &mytid)))

       {DEBUG("Error " <<retc <<" starting MonProc thread.");}

       else {DEBUG("Thread " <<mytid <<" monitoring process " <<monPID);}

    return;

@@ -154,7 +154,7 @@ void bbcp_ProcMon::Start(int seclim, bbcp_BuffPool *bu

 // Run a thread to start the monitor

 //

-   if (retc = bbcp_Thread_Run(bbcp_MonProc, (void *)this, &mytid))

+   if ((retc = bbcp_Thread_Run(bbcp_MonProc, (void *)this, &mytid)))

       {DEBUG("Error " <<retc <<" starting buffpool monitor thread.");}

       else {DEBUG("Thread " <<mytid <<" monitoring buffpool.");}

    return;

--- src/bbcp_ProgMon.C.orig	2015-01-14 20:04:40 UTC

+++ src/bbcp_ProgMon.C

@@ -146,7 +146,7 @@ void bbcp_ProgMon::Start(bbcp_File *fs_obj, bbcp_ZCX *

 // Run a thread to start the monitor

 //

-   if (retc = bbcp_Thread_Run(bbcp_MonProg, (void *)this, &mytid))

+   if ((retc = bbcp_Thread_Run(bbcp_MonProg, (void *)this, &mytid)))

       {DEBUG("Error " <<retc <<" starting progress monitor thread.");}

       else {DEBUG("Thread " <<mytid <<" monitoring progress.");}

    return;

--- src/bbcp_Protocol.C.orig	2015-01-14 20:04:31 UTC

+++ src/bbcp_Protocol.C

@@ -86,7 +86,7 @@ int bbcp_Protocol::Schedule(bbcp_Node *Fnode, bbcp_Fil

 // Start-up the first node

 //

-   if (retc = Fnode->Run(Ffs->username, Ffs->hostname, Fcmd, Ftype))

+   if ((retc = Fnode->Run(Ffs->username, Ffs->hostname, Fcmd, Ftype)))

       return retc;

 // Determine additional options

@@ -101,15 +101,15 @@ int bbcp_Protocol::Schedule(bbcp_Node *Fnode, bbcp_Fil

 // Send the arguments

 //

-   if (retc = SendArgs(Fnode, Ffs, (char *)"none", 0, addOpt[0])) return retc;

+   if ((retc = SendArgs(Fnode, Ffs, (char *)"none", 0, addOpt[0]))) return retc;

 // Get the callback port from the first host

 //

-   if (retc = getCBPort(Fnode)) return retc;

+   if ((retc = getCBPort(Fnode))) return retc;

 // Start the second node

 //

-   if (retc = Lnode->Run(Lfs->username, Lfs->hostname, Lcmd, Ltype))

+   if ((retc = Lnode->Run(Lfs->username, Lfs->hostname, Lcmd, Ltype)))

       return retc;

 // Compute callback hostname and reset callback port

@@ -152,7 +152,7 @@ int bbcp_Protocol::getCBPort(bbcp_Node *Node)

 // The remote program should hve started, get the call back port

 //

-   if (wp = Node->GetLine())

+   if ((wp = Node->GetLine()))

       {if ((wp = Node->GetToken()) && !strcmp(wp, "200")

        &&  (wp = Node->GetToken()) && !strcmp(wp, "Port:")

        &&  (wp = Node->GetToken())

@@ -408,7 +408,7 @@ int bbcp_Protocol::Process_get()

 // Get the optional offset

 //

-   if (wp = Remote->GetToken())

+   if ((wp = Remote->GetToken()))

       {if (bbcp_Config.a2ll("file offset", wp, foffset, 0, -1)) return 22;

        if (foffset > fp->Info.size)

           {char buff[128];

@@ -447,7 +447,7 @@ int bbcp_Protocol::Process_login(bbcp_Link *Net)

 // Get the first line of the login stream

 //

    if (!(np->GetLine()))

-      {if (retc = np->LastError())

+      {if ((retc = np->LastError()))

           return bbcp_Emsg("Process_Login", retc, "processing login from",

                                  Net->LinkName());

        return bbcp_Fmsg("Process_Login", "Bad login from", Net->LinkName());

@@ -549,8 +549,8 @@ int bbcp_Protocol::Request(bbcp_Node *Node)

    if (texists && bbcp_Config.snkSpec->Info.Otype == 'd')

        tdir = bbcp_Config.snkSpec->pathname;

       else {int plen;

-            if (plen = bbcp_Config.snkSpec->filename -

-                       bbcp_Config.snkSpec->pathname)

+            if ((plen = bbcp_Config.snkSpec->filename -

+                       bbcp_Config.snkSpec->pathname))

                strncpy(buff, bbcp_Config.snkSpec->pathname, plen-1);

                else {buff[0] = '.'; plen = 2;}

             tdir = buff; buff[plen-1] = '\0';

@@ -562,7 +562,7 @@ int bbcp_Protocol::Request(bbcp_Node *Node)

       tdir_id = bbcp_Config.snkSpec->Info.fileid;

       else {bbcp_FileInfo Tinfo;

             if (!fs_obj || (!(retc = fs_obj->Stat(tdir, &Tinfo))

-            && Tinfo.Otype != 'd') && outDir) retc = ENOTDIR;

+            && (Tinfo.Otype != 'd') && outDir)) retc = ENOTDIR;

             if (retc) {bbcp_Fmsg("Request","Target directory",

                                  bbcp_Config.snkSpec->pathname,"not found");

                        return Request_exit(2, dRM);

@@ -925,7 +925,7 @@ void bbcp_Protocol::putCSV(char *Host, char *csFn, cha

 {                                //1234567890123

    struct iovec iov[] = {{(char *)"Checksum: ", 10},

                         {bbcp_Config.csName,strlen(bbcp_Config.csName)},

-                        {(char *)" ", 1}, {csVal, csVsz},

+                        {(char *)" ", 1}, {csVal, (unsigned long)csVsz},

                         {(char *)" ", 1}, {Host, strlen(Host)},

                         {(char *)":", 1}, {csFn, strlen(csFn)},

                         {(char *)"\n",1}};

--- src/bbcp_Pthread.C.orig	2015-01-14 20:04:21 UTC

+++ src/bbcp_Pthread.C

@@ -128,7 +128,7 @@ int bbcp_CondVar::WaitMS(int msec)

 /*                              C o n d W a i t                               */

 /******************************************************************************/

-#if defined(MACOS) || defined(AIX)

+#if defined(MACOS) || defined(AIX) || defined(FREEBSD)

 int bbcp_Semaphore::CondWait()

 {

@@ -215,7 +215,7 @@ int  bbcp_Thread_Start(void *(*proc)(void *), void *ar

 void *bbcp_Thread_Wait(pthread_t tid)

     {long retc;

      void *tstat;

-     if (retc = pthread_join(tid, (void **)&tstat)) tstat = (void *)retc;

+     if ((retc = pthread_join(tid, (void **)&tstat))) tstat = (void *)retc;

      return tstat;

     }

--- src/bbcp_Pthread.h.orig	2015-01-14 20:04:16 UTC

+++ src/bbcp_Pthread.h

@@ -108,7 +108,7 @@ void    UnLock() {if (monMutex) {monMutex->UnLock(); m

 bbcp_Mutex *monMutex;

 };

-#if defined(MACOS) || defined(AIX)

+#if defined(MACOS) || defined(AIX) || defined(FREEBSD)

 class bbcp_Semaphore

 {

 public:

--- src/bbcp_Stream.C.orig	2015-01-14 20:05:36 UTC

+++ src/bbcp_Stream.C

@@ -206,7 +206,7 @@ int bbcp_Stream::Exec(char **parm, int inrd, int inerr

            return bbcp_Emsg("Exec",errno,"creating a pipe for",parm[0]);

         Child_In=fildes_In[0]; Child_Out=fildes_Out[1]; Child_Err=fildes_Err[1];

         fildes_Out[1] = (inrd ? fildes_In[1] : -1);

-        if (retc = Attach(fildes_Out)) return retc;

+        if ((retc = Attach(fildes_Out))) return retc;

        }

     // Fork a process first so we can pick up the next request.

@@ -436,7 +436,7 @@ char *bbcp_Stream::GetWord(int lowcase)

      // If we have a token, return it

      //

      xline = 1;

-     if (wp = GetToken(lowcase)) return wp;

+     if ((wp = GetToken(lowcase))) return wp;

      // If no continuation allowed, return a null (but only once)

      //

--- src/bbcp_System.C.orig	2015-01-14 20:05:26 UTC

+++ src/bbcp_System.C

@@ -108,7 +108,7 @@ gid_t bbcp_System::getGID(const char *group)

 // Convert the group name to a gid

 //

    Glookup.Lock();

-   if (gp = getgrnam(group)) gid = gp->gr_gid;

+   if ((gp = getgrnam(group))) gid = gp->gr_gid;

       else gid = (gid_t)-1;

    Glookup.UnLock();

    return gid;

@@ -126,7 +126,7 @@ char *bbcp_System::getGNM(gid_t gid)

 // Get the group name

 //

    Glookup.Lock();

-   if (gp = getgrgid(gid)) gnmp = gp->gr_name;

+   if ((gp = getgrgid(gid))) gnmp = gp->gr_name;

       else gnmp = (char *)"nogroup";

    Glookup.UnLock();

@@ -151,7 +151,7 @@ char *bbcp_System::getHomeDir()

 // Get the password entry for this uid

 //

-   if (pwp = getpwuid(myuid)) homedir = pwp->pw_dir;

+   if ((pwp = getpwuid(myuid))) homedir = pwp->pw_dir;

       else homedir = (char *)"/tmp";

 // Return a copy of the directory

@@ -170,7 +170,7 @@ pid_t bbcp_System::getGrandP()

     int rc, grandpa;

     sprintf(cmd, PS_CMD, getppid());

-    if (rc = cmdstream.Exec(cmd)) rc = cmdstream.LastError();

+    if ((rc = cmdstream.Exec(cmd))) rc = cmdstream.LastError();

        else if (!cmdstream.GetLine() || !(lp = cmdstream.GetLine())) rc = -33;

                else {errno = 0;

                      grandpa = strtol(lp, (char **)NULL, 10);

@@ -218,7 +218,7 @@ char *bbcp_System::UserName()

 // Get the password entry for this uid

 //

-   if (pwp = getpwuid(myuid)) uname = pwp->pw_name;

+   if ((pwp = getpwuid(myuid))) uname = pwp->pw_name;

       else uname = (char *)"anonymous";

 // Return a copy of the directory

--- src/bbcp_ZCX.C.orig	2015-01-14 20:06:03 UTC

+++ src/bbcp_ZCX.C

@@ -144,7 +144,7 @@ int bbcp_ZCX::Process()

 // If we have gotten here then all went well so far flush output

 //

-   if (obp->blen = outsz - ZStream.avail_out)

+   if ((obp->blen = outsz - ZStream.avail_out))

       {obp->boff = outbytes; outbytes += obp->blen;

        Obuff->putFullBuff(obp);

        if (!(obp = Obuff->getEmptyBuff())) return ENOBUFS;