msQueryByPoint does not always free the resultcache of the pevious query

[szekerest]

msQueryByPoint does not free the resultcache of the pevious query if the layer 
is not queryable, therefore the previous results will be reported as the new 
results.


Tamas Szekeres

[szekerest]

Steve,

You should place msIsLayerQueryable one step later to fix this issue as
(mapquery.c 834):

if(lp->resultcache) {
      if(lp->resultcache->results) free(lp->resultcache->results);
      free(lp->resultcache);
      lp->resultcache = NULL;
    }

	if(!msIsLayerQueryable(lp)) continue;

Tamas Szekeres

[sdlime]

In what situation does a layer go from queryable to not queryable? I'm curious.

Note that if this really can be a problem then a couple other of the query
function suffer too.

Steve

[sdlime]

I have made the changes to CVS HEAD, but have not back ported to 4.8 yet. Is
necessaru? Marking as fixed. 

[szekerest]

This situation may occur with long running processes like a desktop solution I
am working on.

Tamas