You can not select more than 25 topics Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.

37 lines
1.3 KiB

  1. From 802d3cbf3043be5dce5317e140ccb1c17a6a2d39 Mon Sep 17 00:00:00 2001
  2. From: Thomas Bernard <miniupnp@free.fr>
  3. Date: Tue, 29 Jan 2019 11:21:47 +0100
  4. Subject: [PATCH] TIFFWriteDirectoryTagTransferfunction() : fix NULL
  5. dereferencing
  6. http://bugzilla.maptools.org/show_bug.cgi?id=2833
  7. we must check the pointer is not NULL before memcmp() the memory
  8. ---
  9. libtiff/tif_dirwrite.c | 6 ++++--
  10. 1 file changed, 4 insertions(+), 2 deletions(-)
  11. diff --git a/libtiff/tif_dirwrite.c b/libtiff/tif_dirwrite.c
  12. index c15a28db..ef30c869 100644
  13. --- a/libtiff/tif_dirwrite.c
  14. +++ b/libtiff/tif_dirwrite.c
  15. @@ -1893,12 +1893,14 @@ TIFFWriteDirectoryTagTransferfunction(TIFF* tif, uint32* ndir, TIFFDirEntry* dir
  16. n=3;
  17. if (n==3)
  18. {
  19. - if (!_TIFFmemcmp(tif->tif_dir.td_transferfunction[0],tif->tif_dir.td_transferfunction[2],m*sizeof(uint16)))
  20. + if (tif->tif_dir.td_transferfunction[2] == NULL ||
  21. + !_TIFFmemcmp(tif->tif_dir.td_transferfunction[0],tif->tif_dir.td_transferfunction[2],m*sizeof(uint16)))
  22. n=2;
  23. }
  24. if (n==2)
  25. {
  26. - if (!_TIFFmemcmp(tif->tif_dir.td_transferfunction[0],tif->tif_dir.td_transferfunction[1],m*sizeof(uint16)))
  27. + if (tif->tif_dir.td_transferfunction[1] == NULL ||
  28. + !_TIFFmemcmp(tif->tif_dir.td_transferfunction[0],tif->tif_dir.td_transferfunction[1],m*sizeof(uint16)))
  29. n=1;
  30. }
  31. if (n==0)
  32. --
  33. 2.18.1