anbuashokcs / jforum2

Automatically exported from code.google.com/p/jforum2
Other
0 stars 0 forks source link

Stack Overflow Error at GenericBanlistDAO.java:126 #25

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Install JForum
2. Don't use it for a couple weeks
3. Pull up JForum in a browser.

What is the expected output? What do you see instead?
Expected the home page, saw very long stackoverflow error instead.

What version of the product are you using? On what operating system?
JForum 2.3.3, Windows7 x64

Please provide any additional information below.
exception

javax.servlet.ServletException: Servlet execution threw an exception
    net.jforum.util.legacy.clickstream.ClickstreamFilter.doFilter(ClickstreamFilter.java:59)
root cause

java.lang.StackOverflowError
    com.mysql.jdbc.Util.handleNewInstance(Util.java:435)
    com.mysql.jdbc.ResultSetImpl.getInstance(ResultSetImpl.java:381)
    com.mysql.jdbc.MysqlIO.buildResultSetWithRows(MysqlIO.java:2653)
    com.mysql.jdbc.MysqlIO.getResultSet(MysqlIO.java:487)
    com.mysql.jdbc.MysqlIO.readResultsForQueryOrUpdate(MysqlIO.java:2631)
    com.mysql.jdbc.MysqlIO.readAllResults(MysqlIO.java:1800)
    com.mysql.jdbc.MysqlIO.sqlQueryDirect(MysqlIO.java:2221)
    com.mysql.jdbc.ConnectionImpl.execSQL(ConnectionImpl.java:2624)
    com.mysql.jdbc.PreparedStatement.executeInternal(PreparedStatement.java:2127)
    com.mysql.jdbc.PreparedStatement.executeQuery(PreparedStatement.java:2293)
    com.mchange.v2.c3p0.impl.NewProxyPreparedStatement.executeQuery(NewProxyPreparedStatement.java:76)
    net.jforum.dao.generic.GenericBanlistDAO.selectAll(GenericBanlistDAO.java:126)
    net.jforum.repository.BanlistRepository.loadBanlist(BanlistRepository.java:134)
    net.jforum.repository.BanlistRepository.banlist(BanlistRepository.java:121)
    net.jforum.repository.BanlistRepository.add(BanlistRepository.java:97)
    net.jforum.repository.BanlistRepository.loadBanlist(BanlistRepository.java:137)
    net.jforum.repository.BanlistRepository.banlist(BanlistRepository.java:121)
    net.jforum.repository.BanlistRepository.add(BanlistRepository.java:97)
    net.jforum.repository.BanlistRepository.loadBanlist(BanlistRepository.java:137)
    net.jforum.repository.BanlistRepository.banlist(BanlistRepository.java:121)
    net.jforum.repository.BanlistRepository.add(BanlistRepository.java:97)
    net.jforum.repository.BanlistRepository.loadBanlist(BanlistRepository.java:137)
    net.jforum.repository.BanlistRepository.banlist(BanlistRepository.java:121)
    net.jforum.repository.BanlistRepository.add(BanlistRepository.java:97)
    net.jforum.repository.BanlistRepository.loadBanlist(BanlistRepository.java:137)
    net.jforum.repository.BanlistRepository.banlist(BanlistRepository.java:121)
    net.jforum.repository.BanlistRepository.add(BanlistRepository.java:97)
    net.jforum.repository.BanlistRepository.loadBanlist(BanlistRepository.java:137)
    net.jforum.repository.BanlistRepository.banlist(BanlistRepository.java:121)
    net.jforum.repository.BanlistRepository.add(BanlistRepository.java:97)
    net.jforum.repository.BanlistRepository.loadBanlist(BanlistRepository.java:137)
    net.jforum.repository.BanlistRepository.banlist(BanlistRepository.java:121)
    net.jforum.repository.BanlistRepository.add(BanlistRepository.java:97)
    net.jforum.repository.BanlistRepository.loadBanlist(BanlistRepository.java:137)
    net.jforum.repository.BanlistRepository.banlist(BanlistRepository.java:121)
    net.jforum.repository.BanlistRepository.add(BanlistRepository.java:97)
    net.jforum.repository.BanlistRepository.loadBanlist(BanlistRepository.java:137)
    net.jforum.repository.BanlistRepository.banlist(BanlistRepository.java:121)
    net.jforum.repository.BanlistRepository.add(BanlistRepository.java:97)
    net.jforum.repository.BanlistRepository.loadBanlist(BanlistRepository.java:137)
    net.jforum.repository.BanlistRepository.banlist(BanlistRepository.java:121)
    net.jforum.repository.BanlistRepository.add(BanlistRepository.java:97)
    net.jforum.repository.BanlistRepository.loadBanlist(BanlistRepository.java:137)
    net.jforum.repository.BanlistRepository.banlist(BanlistRepository.java:121)
    net.jforum.repository.BanlistRepository.add(BanlistRepository.java:97)
    net.jforum.repository.BanlistRepository.loadBanlist(BanlistRepository.java:137)
    net.jforum.repository.BanlistRepository.banlist(BanlistRepository.java:121)
    net.jforum.repository.BanlistRepository.add(BanlistRepository.java:97)
    net.jforum.repository.BanlistRepository.loadBanlist(BanlistRepository.java:137)
    net.jforum.repository.BanlistRepository.banlist(BanlistRepository.java:121)
    net.jforum.repository.BanlistRepository.add(BanlistRepository.java:97)
    net.jforum.repository.BanlistRepository.loadBanlist(BanlistRepository.java:137)
    net.jforum.repository.BanlistRepository.banlist(BanlistRepository.java:121)
    net.jforum.repository.BanlistRepository.add(BanlistRepository.java:97)
    net.jforum.repository.BanlistRepository.loadBanlist(BanlistRepository.java:137)
    net.jforum.repository.BanlistRepository.banlist(BanlistRepository.java:121)
    net.jforum.repository.BanlistRepository.add(BanlistRepository.java:97)
    net.jforum.repository.BanlistRepository.loadBanlist(BanlistRepository.java:137)
    net.jforum.repository.BanlistRepository.banlist(BanlistRepository.java:121)
    net.jforum.repository.BanlistRepository.add(BanlistRepository.java:97)
    net.jforum.repository.BanlistRepository.loadBanlist(BanlistRepository.java:137)
    net.jforum.repository.BanlistRepository.banlist(BanlistRepository.java:121)
    net.jforum.repository.BanlistRepository.add(BanlistRepository.java:97)
    net.jforum.repository.BanlistRepository.loadBanlist(BanlistRepository.java:137)
    net.jforum.repository.BanlistRepository.banlist(BanlistRepository.java:121)
    net.jforum.repository.BanlistRepository.add(BanlistRepository.java:97)
    net.jforum.repository.BanlistRepository.loadBanlist(BanlistRepository.java:137)
    net.jforum.repository.BanlistRepository.banlist(BanlistRepository.java:121)
    net.jforum.repository.BanlistRepository.add(BanlistRepository.java:97)
    net.jforum.repository.BanlistRepository.loadBanlist(BanlistRepository.java:137)
    net.jforum.repository.BanlistRepository.banlist(BanlistRepository.java:121)
    net.jforum.repository.BanlistRepository.add(BanlistRepository.java:97)
    net.jforum.repository.BanlistRepository.loadBanlist(BanlistRepository.java:137)
    net.jforum.repository.BanlistRepository.banlist(BanlistRepository.java:121)
    net.jforum.repository.BanlistRepository.add(BanlistRepository.java:97)
    net.jforum.repository.BanlistRepository.loadBanlist(BanlistRepository.java:137)
    net.jforum.repository.BanlistRepository.banlist(BanlistRepository.java:121)
    net.jforum.repository.BanlistRepository.add(BanlistRepository.java:97)
    net.jforum.repository.BanlistRepository.loadBanlist(BanlistRepository.java:137)
    net.jforum.repository.BanlistRepository.banlist(BanlistRepository.java:121)
    net.jforum.repository.BanlistRepository.add(BanlistRepository.java:97)
    net.jforum.repository.BanlistRepository.loadBanlist(BanlistRepository.java:137)
    net.jforum.repository.BanlistRepository.banlist(BanlistRepository.java:121)
    net.jforum.repository.BanlistRepository.add(BanlistRepository.java:97)
    net.jforum.repository.BanlistRepository.loadBanlist(BanlistRepository.java:137)
    net.jforum.repository.BanlistRepository.banlist(BanlistRepository.java:121)
    net.jforum.repository.BanlistRepository.add(BanlistRepository.java:97)
    net.jforum.repository.BanlistRepository.loadBanlist(BanlistRepository.java:137)
    net.jforum.repository.BanlistRepository.banlist(BanlistRepository.java:121)
    net.jforum.repository.BanlistRepository.add(BanlistRepository.java:97)
    net.jforum.repository.BanlistRepository.loadBanlist(BanlistRepository.java:137)
    net.jforum.repository.BanlistRepository.banlist(BanlistRepository.java:121)
    net.jforum.repository.BanlistRepository.add(BanlistRepository.java:97)
    net.jforum.repository.BanlistRepository.loadBanlist(BanlistRepository.java:137)
    net.jforum.repository.BanlistRepository.banlist(BanlistRepository.java:121)
    net.jforum.repository.BanlistRepository.add(BanlistRepository.java:97)
    net.jforum.repository.BanlistRepository.loadBanlist(BanlistRepository.java:137)
    net.jforum.repository.BanlistRepository.banlist(BanlistRepository.java:121)
    net.jforum.repository.BanlistRepository.add(BanlistRepository.java:97)
    net.jforum.repository.BanlistRepository.loadBanlist(BanlistRepository.java:137)
    net.jforum.repository.BanlistRepository.banlist(BanlistRepository.java:121)
    net.jforum.repository.BanlistRepository.add(BanlistRepository.java:97)
    net.jforum.repository.BanlistRepository.loadBanlist(BanlistRepository.java:137)

------------snip several hundred similar lines--------------

    net.jforum.repository.BanlistRepository.banlist(BanlistRepository.java:121)
    net.jforum.repository.BanlistRepository.add(BanlistRepository.java:97)
    net.jforum.repository.BanlistRepository.loadBanlist(BanlistRepository.java:137)
note The full stack trace of the root cause is available in the Apache 
Tomcat/6.0.20 logs.

Original issue reported on code.google.com by pete.lam...@gmail.com on 16 Mar 2012 at 9:52

GoogleCodeExporter commented 9 years ago
There's a loop in the method call sequence:
loadBanlist()->add()->banlist()->loadBanlist()
Should be break in banlist()

Original comment by andow...@gmail.com on 21 May 2012 at 9:12

GoogleCodeExporter commented 9 years ago
Fixed in revision r184

Original comment by andow...@gmail.com on 23 May 2012 at 8:55