Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Mapserver hangs if the text is empty #5167

Closed
krzbor opened this issue Sep 19, 2015 · 5 comments
Closed

Mapserver hangs if the text is empty #5167

krzbor opened this issue Sep 19, 2015 · 5 comments

Comments

@krzbor
Copy link

krzbor commented Sep 19, 2015

Mapserver (7.0.0) hangs if the text is empty:
LAYER
#NAME je_100_n
GROUP granice
METADATA
"wms_title" "Powiaty - nazwy" ##required‚‚
END
PROJECTION
"init=epsg:2180" ##required
END

DATA         je100O
STATUS       off
TYPE         polygon
LABELITEM    "NAZWA" #  <--------- hangs if the text is empty
CLASS
    LABEL
        ENCODING WINDOWS-1250
        FORCE TRUE
        COLOR 200 0 0
        TYPE TRUETYPE
        FONT arial
        SIZE 9
        ANTIALIAS FALSE
        POSITION cc
        PARTIALS FALSE
        STYLE
            GEOMTRANSFORM 'labelpoly'
            COLOR 255 255 255
        END
    END # end of label
END

END

@tbonfort
Copy link
Member

unable to reproduce with a simple testcase. please provide a full testcase (data + full mapfile + request)

@krzbor
Copy link
Author

krzbor commented Sep 21, 2015

It is difficult for me to extract a small piece of data and file maps. The server hangs when I try to view the area where the attribute value is empty. I changed a blank text on the minus sign (inside DBF file) and everything was OK.

@mkofahl
Copy link
Contributor

mkofahl commented Sep 21, 2015

@krzbor Are you able to compile branch-7-0 with patch 5570f5d yourself? I noticed the same behaviour (infinitve loop until http server timeout) when labelitem met database null-values. A simplified test-case worked however. I found a small typo in msAddLabelGroup which solved the problem for mee.

@krzbor
Copy link
Author

krzbor commented Sep 23, 2015

5570f5d patch solved my problem. Thank you mkofahl

@ejn
Copy link
Contributor

ejn commented Oct 12, 2015

Merged #5169 fixes this according to OP. Closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants