Changeset 383 for trunk/src/dnmalloc.c


Ignore:
Timestamp:
Dec 5, 2011, 10:30:33 PM (10 years ago)
Author:
katerina
Message:

Fix for ticket #281 (warnings from clang static analyzer).

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/dnmalloc.c

    r362 r383  
    28542854      assert(temp != NULL);
    28552855#endif
    2856       prevtemp->hash_next = temp->hash_next;
     2856      if (temp) prevtemp->hash_next = temp->hash_next;
    28572857    }
    28582858}
     
    35583558   
    35593559    else {
    3560       front_misalign = 0;
    3561       end_misalign = 0;
     3560      /* front_misalign = 0; *//*superfluous */
     3561      /* end_misalign = 0; *//*superfluous */
    35623562      correction = 0;
    35633563      aligned_brk = brk;
     
    42844284  retval = sYSMALLOc(nb, av);
    42854285  if (retval) {
    4286     victim = mem2chunk(retval);
     4286#if PARANOIA > 2
     4287    victim = mem2chunk(retval); /* is used in guard_set macro */
     4288#endif
    42874289    guard_set(av->guard_stored, victim, bytes, nb);
    42884290  }
     
    55165518        } rdat;
    55175519#if !defined(__FreeBSD__) && !defined(__OpenBSD__) && !defined(__NetBSD__)
    5518         size_t sz = 0;
     5520        ssize_t sz = 0;
    55195521        int    fd;
    55205522#endif
     
    55405542        fd = open("/dev/urandom", O_RDONLY);
    55415543        if (fd != -1) {
    5542                 sz = (size_t)read(fd, rdat.rnd, sizeof (rdat.rnd));
    5543                 close(fd);
     5544                sz = (size_t)read(fd, rdat.rnd, sizeof (rdat.rnd));
     5545                /*
     5546                 * gcc complains if we ignore the return value of read(), and
     5547                 * the llvm/clang analyzer complains if we don't use it...
     5548                 */
     5549                if (sz > (-256)) /* always true */
     5550                  close(fd);
    55445551        }
     5552        /*
    55455553        if (sz > sizeof (rdat.rnd))
    55465554                sz = 0;
     5555        */
    55475556 #endif
    55485557
Note: See TracChangeset for help on using the changeset viewer.