FreeTDS Development Group

[Steve Willer <willer AT novator.com>]

On Wed, 20 Mar 2002, James Cameron wrote:

> Steve Willer wrote:
> > In this patch, shouldn't the line:
> >    if (thisgot <= 0) break;
> > actually be:
> >    if (thisgot <= 0) return -1;
> 
> That's exactly what I asked.  ;-)

Ah.

Actually, when I thought about this a little further, I realized that you
might want to have retries on EINTR error conditions.This would allow you
to continue processing even if you've received unrelated SIGALRM signals,
or a SIGHUP in an Apache environment, etc.

  if (thisgot == 0) return got;
  if (thisgot < 0) {
     if (errno == EINTR) return 0;
     return -1;
  }