I totally agree to Lincoln’s suggestion. Don’t use <c:if> in your applications and instead use the rendered attribute. I cannot imagine a single case where <c:if> should be preferred. Actually I never used it in any of my application. The NPE you are seeing is probably related to something else.
Do you play Magic?
Get TopDecked MTG, the #1 Magic App, built by players, for players. Free to use, forever.