<br><font size=2 face="sans-serif">hi  frank</font>
<br>
<br><font size=2 face="sans-serif">i did it -> bug 1403</font>
<br>
<br><font size=2 face="sans-serif">bye</font>
<br><font size=2 face="sans-serif">benedikt</font>
<br>
<br><font size=2><tt>Frank Warmerdam <fwarmerdam@gmail.com> schrieb
am 29.06.2005 16:16:16:<br>
<br>
> On 6/29/05, Benedikt Rothe <umn-ms@hydrotec.de> wrote:<br>
> >  <br>
> > Dear List <br>
> >  <br>
> > We are using MapScript/Tomcat/Java/OracleSpatial. <br>
> >  <br>
> > We use mapObj.mapClone() to get a fresh mapObject <br>
> > in each request without reading the Mapfile. <br>
> >  <br>
> > We detected problems after we activated Connection-Pooling: <br>
> > Tomcat crashed in mapObj.delete() <br>
> >  <br>
> > Hole thing turned out to be a bug in mapcopy.c where <br>
> > the joins are copied: <br>
> > -------------------- <br>
> > MS_COPYSTELEM(numjoins); <br>
> >  <br>
> > for (i = 0; i < dst->numprocessing; i++) { <br>
> >    return_value = msCopyJoin(&(dst->joins[i]),
&(src->joins[i])); <br>
> > -------------------- <br>
> >  <br>
> > This must be changed to <br>
> > -------------------- <br>
> > MS_COPYSTELEM(numjoins); <br>
> >  <br>
> > for (i = 0; i < dst->numjoins; i++) { <br>
> >    return_value = msCopyJoin(&(dst->joins[i]),
&(src->joins[i])); <br>
> > -------------------- <br>
> >  <br>
> > There is some other code in mapobject.c, wich didn't <br>
> > raise any problems for us, but from wich I do think <br>
> > that array-allocs are missing. <br>
> > I watched the code an *think* these are are not allocated <br>
> > anywhere. I hope, there is no misleading hint: <br>
> <br>
> Benedikt, <br>
> <br>
> I would encourage you to submit this via bugzilla, <br>
> and ensure it gets assigned to Sean Gilles who<br>
> takes care of map cloning problems. <br>
> <br>
> The cloning is a bit prone to bugs, but we want to fix it<br>
> up. <br>
> <br>
> Best regards,<br>
> -- <br>
> ---------------------------------------+--------------------------------------<br>
> I set the clouds in motion - turn up   | Frank Warmerdam, warmerdam@pobox.com<br>
> light and sound - activate the windows | http://pobox.com/~warmerdam<br>
> and watch the world go round - Rush    | Geospatial Programmer
for Rent<br>
</tt></font>