<div class="gmail_quote">On Wed, Nov 3, 2010 at 02:39, Andreas Hauffe <span dir="ltr"><<a href="mailto:andreas.hauffe@tu-dresden.de">andreas.hauffe@tu-dresden.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
Hi,<br>
<br>
during the compilation process I'm getting the following warning. Is this a<br>
problem?<br>
<br>
Warning libfast in: /scratch/ahauffe/petsc-3.1-p5/src/vec/vec/utils<br>
vscat.c(322): warning #592: variable "sto" is used before its value is set<br>
ierr =<br>
PetscMalloc2(1,VecScatter_MPI_ToAll,&sto,size,PetscMPIInt,&sto-<br>
>count);CHKERRQ(ierr);</blockquote><div><br></div><div>Thanks for pointing this out, I didn't notice when it was introduced (to fix a different warning).</div><div><br></div><div><a href="http://petsc.cs.iit.edu/petsc/petsc-dev/rev/4ff7b4bc0bea">http://petsc.cs.iit.edu/petsc/petsc-dev/rev/4ff7b4bc0bea</a></div>
<div><br></div><div>I'm surprised other compilers haven't warned about this, and that we haven't seen any bad behavior because of it.</div><div><br></div><div>We either need to find a way to tweak the macro to clean up after itself correctly in the case of malloc failure, without dereferencing something that may not exist yet (conflicting goals), or fix any calls that try to build a structure like this with a single malloc.</div>
<div><br></div><div>Jed</div></div>