vtkKdNode, vtkOctreePointLocatorNode: don't assign after a short-circuit

Conditionals may short-circuit and assignments anywhere other than the
first sub-expression is not guaranteed to be executed, so just always do
the subtraction.

Change-Id: I8e1ed5ad4f1ac3257209ee0386c5965cfd8d1e2f
vtkPolyhedron: initialize the iterator

In the loop, the map can cause the current iterator in cpSetIt to be
erased from cpSet, but it is not guaranteed to be initialized.

Change-Id: I793dc4b50c0ffcc616f59b9550c507e8f5e04479
HyperTree example: use the iterator before invalidating it

Change-Id: I88b20ac27e198a759d1c9a1c8d9e4b6ddc1d3da6
Widgets example: fix up logic around planeSource

Change-Id: I5bf5a76e0d57c2a33ffd8f95f63e83bdf27313fd
vtkYoungsMaterialInterface: plug memory leaks

Change-Id: I6fcdd2e8faf210bf7dadc8bafb49a1d38ac23f63
vtkUnstructuredGridGeometryFilter: plug memory leak

Change-Id: I84a9f361ed120c4a46a308dd772a2d5df259811b
vtkPKdTree: plug memory leak

Change-Id: I73c6692d5eb0585d7f6504cad3f2c74531d5fdd0
vtkDistributedDataFilter: plug memory leaks

Change-Id: If46406100cb9dddd762aa4353065c8dbde9aae49
vtkKMeansStatistics: plug memory leak

Change-Id: I78895a6ca1841d2177efeef289d3642b0a9a1257
vtkAMRFlashReaderInternal: don't return a pointer to temporary memory

The string only lives as long as its expression, so don't return its
internal c_str.

Change-Id: I32f9fc8ef9847baa645d5743d399b55782122114
vtkExodusIIWriter: use a larger buffer

Missing space for a NUL byte.

Change-Id: I130ca197de1c37c087884b27be30e50de7e9909b
TestIncrementalOctreePointLocator: close file handles

Change-Id: I98c0548b968ad06d97373d89d0e375cd71a058c0
vtkCGMWriter: fix typo

Change-Id: Ic781ec4a635566d7a25aa071910cb8516888c3c4
vtkNIFTIImageReader: plug memory leaks

Change-Id: I96903ef5343f319a4e33e84f0afaaf709cfb8753
vtkSLCReader: plug memory leak

Change-Id: I94a508d159219f833d6363ae1c9078c94aa06c47
vtkPLY: handle realloc errors properly

Change-Id: Ie2cf9e3d5848a22120899a3f867b45808f0f2e78
vtkEnSightWriter: terminate buffers properly

strncat doesn't guarantee a NUL-terminated string after it is done.

Change-Id: I4f1fd122e928f529a7e363e30d4c708be7a132f3
vtkImagePlaneWidget: avoid divide-by-zero

Change-Id: I136a04d90cd32c36960a8c1c4f25efb7d25b66b2
vtkResliceCursorRepresentation: avoid divide-by-zero

Change-Id: Id533f580ce894f8e6c7d17943259c455ec1dfe7d
vtkParseMain: handle realloc errors properly

Change-Id: I94a75a4b4e3eb6d8890160dae9bbcbbdb27be27c