cipher: Remove useless NULL check.
authorWerner Koch <wk@gnupg.org>
Mon, 16 Mar 2015 08:01:24 +0000 (09:01 +0100)
committerWerner Koch <wk@gnupg.org>
Mon, 16 Mar 2015 08:31:03 +0000 (09:31 +0100)
* cipher/hash-common.c (_gcry_md_block_write): Remove NUL check for
hd->buf.
--

HD->BUF is not allocated but part of the struct.  HD has already be
dereferenced twice thus the check does not make sense.  Detected by
Stack 0.3:

  bug: anti-simplify
  model: |
    %cmp4 = icmp eq i8* %arraydecay, null, !dbg !29
    -->  false
  stack:
    - /home/wk/s/libgcrypt/cipher/hash-common.c:114:0
  ncore: 1
  core:
    - /home/wk/s/libgcrypt/cipher/hash-common.c:108:0
      - null pointer dereference

Signed-off-by: Werner Koch <wk@gnupg.org>
cipher/hash-common.c

index 9a007e1..6743f09 100644 (file)
@@ -111,7 +111,7 @@ _gcry_md_block_write (void *context, const void *inbuf_arg, size_t inlen)
   if (sizeof(hd->buf) < blocksize)
     BUG();
 
-  if (hd->buf == NULL || hd->bwrite == NULL)
+  if (!hd->bwrite)
     return;
 
   if (hd->count == blocksize)  /* Flush the buffer. */