Здравствуйте, Аноним, Вы писали:

..
А>Вот собственно и все. Мне кажется это не из-за bind т.к. на XP+98 все работает корректно. А в XP+XP (правда только в тесте участвовали 3 машины XP) может преслать либо некоторое кол-во байт а потом зайти в send и от туда не выйти, может вообще не переслать, а случалось и пересылал весь файл.

А>ЗЫ: Такое чуйство что в send`е идет блок. Компилятор C++ Builder 6
А>ЗЗЫ: ПЛЗ!!! =)) Не могли бы вы отправить мне на мыло ваш пример просто из спортивного интереса =) ynblpb@hotbox.ru или ynblpb@freemail.ru
А>можть и я вам когда-нибудь помогу
А>

1. Думается, это может происходить из-за того, что recv может возвращать не все запрошенные данные, а по частям (также и send) — пример: послали строку "Вперёд, комсомол!", а получаем так: "Вперёд" ", " "комсо" "мол!", т.е. за четыре вызова recv. Т.е. нужен цикл, пока всё не считаем. Примеров не нашёл, есть код из проекта:

long
TSockHelper::Recv(SOCKET sock, void *Buf_, int iSize, int iTimeout)
{
  TIMEVAL TheTime={1, 0};
  int err;
  int rc;
  fd_set readfds;
  fd_set exceptfds;
  FD_ZERO(&readfds);
  FD_ZERO(&exceptfds);
  int iShift=0;
  char *Buf=(char*)Buf_;

  while(iSize > 0)
   {
    FD_SET(sock, &readfds);
    FD_SET(sock, &exceptfds);

    err=::select(0, &readfds, 0, &exceptfds, &TheTime);

    if(err == SOCKET_ERROR || FD_ISSET(sock, &exceptfds))
      return SOCKET_ERROR;

    if(FD_ISSET(sock, &readfds))
     {
      rc=::recv(sock, Buf+iShift, iSize, 0);
      if(rc == SOCKET_ERROR)
        return SOCKET_ERROR;
      iShift+=rc;
      iSize-=rc;
    }
    else
    if(err == 0)
     {
      if(iTimeout < 0 && --iTimeout == 0)
        return SOCKET_ERROR;
    }
  }

  return iShift;
}

//вызов
struct MyData{
...
};

MyData data;
TSockHelper::Recv(sock, &data, sizeof(data), 10); //10 seconds timeout

аналогично и для send написана Send.

2. Потенциальные ошибки:
из п.1 следует, что здесь в buff могло ещё не приняться sizeof(PckHd) байт, поэтому ((PckHd*)buff)->CurrPacketSize) вернёт неверное значение:

         realloc(buff,sizeof(PckHd)+((PckHd*)buff)->CurrPacketSize);
         recv(s,buff,sizeof(buff),0);

функция realloc переразмещает буфер,

The size argument gives the new size of the block, in bytes. The contents of the block are unchanged up to the shorter of the new and old sizes, although the new block can be in a different location. Because the new block can be in a new memory location, the pointer returned by realloc is not guaranteed to be the pointer passed through the memblock argument.

поэтому верное использование такое:

         buff=realloc(buff,sizeof(PckHd)+((PckHd*)buff)->CurrPacketSize);

2.1 recv(s,buff,sizeof(buff),0); — как думаешь чему равно sizeof(buff)? правильно — всего 4 байта (32 бита) под Win32, поэтому здесь принимаешь максимум 4 байта. Думаю, ты хотел написать примерно следующее:

         int rc=recv(s, buff+sizeof(PckHd), ((PckHd*)buff)->CurrPacketSize,0);

написал buff+sizeof(PckHd), потому что заголовок-то уже получен (хотя, может, не совсем понял протокола — не настаиваю )

2.2. err=FileWrite(hFile,(char*)buff+sizeof(PckHd),((PckHd*)buff)->CurrPacketSize);
соответственно и в файл писать надо, то количество байт, которое вернул recv.

Код посмотрел не полностью (большой больно). Но, думаю, что и так чуть-чуть поможет .