Changeset 4232


Ignore:
Timestamp:
01/06/10 03:00:41 (3 years ago)
Author:
sam
Message:

Fix a bug caused by our seek() not understanding eglibc's behaviour.

Location:
zzuf/trunk/src
Files:
2 edited

Legend:

Unmodified
Added
Removed

  • zzuf/trunk/src/common/fuzz.c

    r4112 r4232  
    9595 
    9696#if defined LIBZZUF 
    97     debug2("fuzz(%i, @%lli, %lli)", fd, (long long int)pos, (long long int)len); 
     97    debug2("... fuzz(%i, @%lli, %lli)", fd, (long long int)pos, 
     98           (long long int)len); 
    9899#endif 
    99100 

  • zzuf/trunk/src/libzzuf/lib-stream.c

    r4230 r4232  
    220220static inline void debug_stream(char const *prefix, FILE *stream) 
    221221{ 
    222     debug2("%p stream([%i], %p, %i)", prefix, fileno(stream), 
    223            get_stream_ptr(stream), get_stream_cnt(stream)); 
     222    debug2("... %s stream([%i], %p, %i + %i)", prefix, fileno(stream), 
     223           get_stream_ptr(stream), get_stream_off(stream), 
     224           get_stream_cnt(stream)); 
    224225} 
    225226 
    … …  
    246247            _zz_register(fd); \ 
    247248            _zz_fuzz(fd, get_stream_ptr(ret), get_stream_cnt(ret)); \ 
     249            debug_stream("after", ret); \ 
    248250            debug("%s(\"%s\", \"%s\") = [%i]", __func__, path, mode, fd); \ 
    249             debug_stream("new", ret); \ 
    250251        } \ 
    251252    } while(0) 
    … …  
    335336        if(!_zz_ready || !_zz_iswatched(fd) || !_zz_isactive(fd)) \ 
    336337            return ORIG(myfseek)(stream, offset, whence); \ 
    337         debug_stream("old", stream); \ 
     338        debug_stream("before", stream); \ 
    338339        /* FIXME: ftell() will return -1 on a pipe such as stdin */ \ 
    339340        oldpos = MYFTELL(stream); \ 
    … …  
    344345        _zz_unlock(fd); \ 
    345346        newpos = MYFTELL(stream); \ 
    346         if (newpos > oldpos + oldcnt || newpos < oldpos - oldoff) \ 
     347        if (newpos >= oldpos + oldcnt || newpos < oldpos - oldoff) \ 
    347348        { \ 
    348349            _zz_setpos(fd, newpos - get_stream_off(stream)); \ 
    … …  
    351352        } \ 
    352353        _zz_setpos(fd, newpos); \ 
     354        debug_stream("after", stream); \ 
    353355        debug("%s([%i], %lli, %s) = %i", __func__, \ 
    354356              fd, (long long int)offset, get_seek_mode_name(whence), ret); \ 
    355         debug_stream("new", stream); \ 
    356357    } while(0) 
    357358 
    … …  
    366367        if(!_zz_ready || !_zz_iswatched(fd) || !_zz_isactive(fd)) \ 
    367368            return ORIG(myfsetpos)(stream, pos); \ 
    368         debug_stream("old", stream); \ 
     369        debug_stream("before", stream); \ 
    369370        /* FIXME: ftell() will return -1 on a pipe such as stdin */ \ 
    370371        oldpos = MYFTELL(stream); \ 
    … …  
    375376        _zz_unlock(fd); \ 
    376377        newpos = MYFTELL(stream); \ 
    377         if (newpos > oldpos + oldcnt || newpos < oldpos - oldoff) \ 
     378        if (newpos >= oldpos + oldcnt || newpos < oldpos - oldoff) \ 
    378379        { \ 
    379380            _zz_setpos(fd, newpos - get_stream_off(stream)); \ 
    … …  
    382383        } \ 
    383384        _zz_setpos(fd, (int64_t)FPOS_CAST(*pos)); \ 
     385        debug_stream("after", stream); \ 
    384386        debug("%s([%i], %lli) = %i", __func__, \ 
    385387              fd, (long long int)FPOS_CAST(*pos), ret); \ 
    386         debug_stream("new", stream); \ 
    387388    } \ 
    388389    while(0) 
    … …  
    398399        if(!_zz_ready || !_zz_iswatched(fd) || !_zz_isactive(fd)) \ 
    399400            return ORIG(rewind)(stream); \ 
    400         debug_stream("old", stream); \ 
     401        debug_stream("before", stream); \ 
    401402        /* FIXME: ftell() will return -1 on a pipe such as stdin */ \ 
    402403        oldpos = MYFTELL(stream); \ 
    … …  
    407408        _zz_unlock(fd); \ 
    408409        newpos = MYFTELL(stream); \ 
    409         if (newpos > oldpos + oldcnt || newpos < oldpos - oldoff) \ 
     410        if (newpos >= oldpos + oldcnt || newpos < oldpos - oldoff) \ 
    410411        { \ 
    411412            _zz_setpos(fd, newpos - get_stream_off(stream)); \ 
    … …  
    414415        } \ 
    415416        _zz_setpos(fd, newpos); \ 
     417        debug_stream("after", stream); \ 
    416418        debug("%s([%i])", __func__, fd); \ 
    417         debug_stream("new", stream); \ 
    418419    } while(0) 
    419420 
    … …  
    483484        if(!_zz_ready || !_zz_iswatched(fd) || !_zz_isactive(fd)) \ 
    484485            return ORIG(myfread)(ptr, size, nmemb, stream); \ 
    485         debug_stream("old", stream); \ 
     486        debug_stream("before", stream); \ 
    486487        /* FIXME: ftell() will return -1 on a pipe such as stdin */ \ 
    487488        oldpos = MYFTELL(stream); \ 
    … …  
    492493        _zz_unlock(fd); \ 
    493494        newpos = MYFTELL(stream); \ 
    494         if (newpos > oldpos + oldcnt) \ 
     495        if (newpos >= oldpos + oldcnt) \ 
    495496        { \ 
    496497            /* Fuzz returned data that wasn't in the old internal buffer */ \ 
    … …  
    503504        } \ 
    504505        _zz_setpos(fd, newpos); \ 
    505         if (newpos > oldpos + 4) \ 
     506        debug_stream("after", stream); \ 
     507        if (newpos >= oldpos + 4) \ 
    506508            debug("%s(%p, %li, %li, [%i]) = %li \"%c%c%c%c...", __func__, \ 
    507509                  ptr, (long int)size, (long int)nmemb, fd, \ 
    … …  
    514516            debug("%s(%p, %li, %li, [%i]) = %li", __func__, ptr, \ 
    515517                  (long int)size, (long int)nmemb, fd, (long int)ret); \ 
    516         debug_stream("new", stream); \ 
    517518    } while(0) 
    518519 
    … …  
    548549        if(!_zz_ready || !_zz_iswatched(fd) || !_zz_isactive(fd)) \ 
    549550            return ORIG(myfgetc)(arg); \ 
    550         debug_stream("old", s); \ 
     551        debug_stream("before", s); \ 
    551552        oldpos = MYFTELL(s); \ 
    552553        oldoff = get_stream_off(s); \ 
    … …  
    564565            ret = ch; \ 
    565566        } \ 
    566         if (newpos > oldpos + oldcnt) \ 
     567        if (newpos >= oldpos + oldcnt) \ 
    567568        { \ 
    568569            /* Fuzz the internal stream buffer */ \ 
    … …  
    572573        } \ 
    573574        _zz_setpos(fd, newpos); \ 
     575        debug_stream("after", s); \ 
    574576        if(ret == EOF) \ 
    575577            debug("%s([%i]) = EOF", __func__, fd); \ 
    576578        else \ 
    577579            debug("%s([%i]) = '%c'", __func__, fd, ret); \ 
    578         debug_stream("new", s); \ 
    579580    } while(0) 
    580581 
    … …  
    643644        if(!_zz_ready || !_zz_iswatched(fd) || !_zz_isactive(fd)) \ 
    644645            return ORIG(myfgets)(s, size, stream); \ 
    645         debug_stream("old", stream); \ 
     646        debug_stream("before", stream); \ 
    646647        oldpos = MYFTELL(stream); \ 
    647648        oldoff = get_stream_off(stream); \ 
    … …  
    670671                    chr = ch; \ 
    671672                } \ 
    672                 if (newpos > oldpos + oldcnt) \ 
     673                if (newpos >= oldpos + oldcnt) \ 
    673674                { \ 
    674675                    /* Fuzz the internal stream buffer, if necessary */ \ 
    … …  
    696697        } \ 
    697698        _zz_setpos(fd, newpos); \ 
     699        debug_stream("after", stream); \ 
    698700        debug("%s(%p, %i, [%i]) = %p", __func__, s, size, fd, ret); \ 
    699         debug_stream("new", stream); \ 
    700701    } while(0) 
    701702 
    … …  
    725726        return ORIG(ungetc)(c, stream); 
    726727 
    727     debug_stream("old", stream); 
     728    debug_stream("before", stream); 
    728729    oldpos = MYFTELL(stream); 
    729730    _zz_lock(fd); 
    … …  
    732733    _zz_setpos(fd, oldpos - 1); 
    733734 
     735    debug_stream("after", stream); 
    734736    if(ret == EOF) 
    735737        debug("%s(0x%02x, [%i]) = EOF", __func__, c, fd); 
    736738    else 
    737739        debug("%s(0x%02x, [%i]) = '%c'", __func__, c, fd, ret); 
    738     debug_stream("new", stream); 
    739740    return ret; 
    740741} 
    … …  
    753754        return ORIG(fclose)(fp); 
    754755 
    755     debug_stream("old", fp); 
     756    debug_stream("before", fp); 
    756757    _zz_lock(fd); 
    757758    ret = ORIG(fclose)(fp); 
    … …  
    780781        if(!_zz_ready || !_zz_iswatched(fd) || !_zz_isactive(fd)) \ 
    781782            return ORIG(getdelim)(lineptr, n, delim, stream); \ 
    782         debug_stream("old", stream); \ 
     783        debug_stream("before", stream); \ 
    783784        oldpos = MYFTELL(stream); \ 
    784785        oldoff = get_stream_off(stream); \ 
    … …  
    812813                chr = ch; \ 
    813814            } \ 
    814             if (newpos > oldpos + oldcnt) \ 
     815            if (newpos >= oldpos + oldcnt) \ 
    815816            { \ 
    816817                /* Fuzz the internal stream buffer, if necessary */ \ 
    … …  
    839840        } \ 
    840841        _zz_setpos(fd, newpos); \ 
     842        debug_stream("after", stream); \ 
    841843        if(need_delim) \ 
    842844            debug("%s(%p, %p, '%c', [%i]) = %li", __func__, \ 
    … …  
    845847            debug("%s(%p, %p, [%i]) = %li", __func__, \ 
    846848                  lineptr, n, fd, (long int)ret); \ 
    847         debug_stream("new", stream); \ 
    848849        break; \ 
    849850    } while(0) 
    … …  
    889890        return ORIG(fgetln)(stream, len); 
    890891 
    891     debug_stream("old", stream); 
     892    debug_stream("before", stream); 
    892893    oldpos = MYFTELL(stream); 
    893894    oldoff = get_stream_off(stream); 
    … …  
    914915            chr = ch; 
    915916        } 
    916         if (newpos > oldpos + oldcnt) 
     917        if (newpos >= oldpos + oldcnt) 
    917918        { 
    918919            /* Fuzz the internal stream buffer, if necessary */ 
    … …  
    940941    ret = fuzz->tmp; 
    941942 
     943    debug_stream("after", stream); 
    942944    debug("%s([%i], &%li) = %p", __func__, fd, (long int)*len, ret); 
    943     debug_stream("new", stream); 
    944945    return ret; 
    945946} 
    … …  
    966967        if(!_zz_ready || !_zz_iswatched(fd) || !_zz_isactive(fd)) \ 
    967968            return ORIG(myrefill)(fp); \ 
    968         debug_stream("old", fp); \ 
     969        debug_stream("before", fp); \ 
    969970        pos = _zz_getpos(fd); \ 
    970971        _zz_lock(fd); \ 
    … …  
    10011002        } \ 
    10021003        _zz_setpos(fd, pos); /* FIXME: do we always need to do this? */ \ 
     1004        debug_stream("after", fp); \ 
    10031005        if (REFILL_RETURNS_INT) \ 
    10041006            debug("%s([%i]) = %i", __func__, fd, ret); \ 
    … …  
    10071009        else \ 
    10081010            debug("%s([%i]) = '%c'", __func__, fd, ret); \ 
    1009         debug_stream("new", fp); \ 
    10101011    } \ 
    10111012    while(0) 
Note: See TracChangeset for help on using the changeset viewer.