requests per child limitation handling bug

1 files changed, 1 insertions, 1 deletions

diff --git a/src/sitecing-fastcgi.cc b/src/sitecing-fastcgi.cc
index d8f18a6..57aba60 100644
--- a/src/sitecing-fastcgi.cc
+++ b/src/sitecing-fastcgi.cc
@@ -89,129 +89,129 @@ sitecing_fastcgi_pm::sitecing_fastcgi_pm(const string& config_file)
             throw konforka::exception(CODEPOINT,"unspecified root for sources");
         if(!(config.flags&configuration::flag_root_intermediate))
             throw konforka::exception(CODEPOINT,"unspecified root for intermediate files");
         if(!(config.flags&configuration::flag_root_so))
             throw konforka::exception(CODEPOINT,"unspecified root for shared objects");
         if(config.flags&configuration::flag_min_children)
             min_children = config.min_children;
         if(config.flags&configuration::flag_max_children)
             max_children = config.max_children;
         if(config.flags&configuration::flag_min_spare_children)
             min_spare_children = config.min_spare_children;
         if(config.flags&configuration::flag_max_spare_children)
             max_spare_children = config.max_spare_children;
         if(max_children<min_spare_children)
             throw konforka::exception(CODEPOINT,"inconsistent numbers of MaxChildren and MinSpareChildren");
         if(min_children>max_spare_children && max_spare_children>=0)
             throw konforka::exception(CODEPOINT,"inconsistent numbers of MinChildren and MaxSpareChildren");
         if(config.flags&configuration::flag_multi_process) {
             multi = config.multi_process;
         }else{
             if(config.flags&configuration::flag_listen_socket)
                 multi = true;
             else
                 multi = false;
         }
         fss = (config.flags&configuration::flag_listen_socket)?new fcgi_socket(config.listen_socket.c_str(),5):new fcgi_socket(0);
         if(!fss)
             throw konforka::exception(CODEPOINT,"failed to establish listening socket");
         if(config.flags&configuration::flag_daemonize && config.daemonize) {
             pid_t pf = fork();
             if(pf<0)
                 throw konforka::exception(CODEPOINT,"failed to fork()");
             if(pf) {
                 die_humbly=true;
                 _exit(0);
             }
         }
         if(config.flags&configuration::flag_pid_file) {
             pidfile.set(config.pid_file);
         }
         if(multi)
             sem.init();
     }
 sitecing_fastcgi_pm::~sitecing_fastcgi_pm() {
     if(fss)
         delete fss;
 }
 
 void sitecing_fastcgi_pm::process(int slot) {
     signal(SIGINT,SIG_DFL);
     signal(SIGABRT,SIG_DFL);
     signal(SIGTERM,SIG_DFL);
     give_up_privs();
     scoreboard_slot *sslot = sboard.get_slot(slot);
     try {
         sitespace ss(config);
         fcgi_socket& fs = *fss;
         sitecing_interface_cgi scif(&ss);
         string component_path;
         string action;
         config_options::action_handler_t *action_handler;
         int rpc = 0;
         if(config.flags&configuration::flag_requests_per_child)
             rpc = config.requests_per_child;
-        for(int req=0;(rpc<=0) || (req<rpc);rpc++) {
+        for(int req=0;(rpc<=0) || (req<rpc);++req) {
             semaphore_lock sl;
             if(multi) {
                 sslot->state = scoreboard_slot::state_idle;
                 sl.sem = &sem;
                 sl.lock();
             }
             sslot->state = scoreboard_slot::state_accept;
             fcgi_interface fi(fs);
             sslot->state = scoreboard_slot::state_processing;
             if(multi)
                 sl.unlock();
             cgi_gateway gw(fi);
             scif.prepare(&gw);
             try {
                 component_path = konforka::normalize_path(gw.path_info(),konforka::strip_leading_slash|konforka::strip_trailing_slash);
                 string full_component_path;
                 string sitecing_path_info;
                 while(true) {
                     full_component_path = config.root_source+'/'+component_path;
                     if(!access(full_component_path.c_str(),F_OK))
                         break;
                     string::size_type sl = component_path.rfind('/');
                     if(sl==string::npos)
                         throw konforka::exception(CODEPOINT,"can't find the target component");
                     sitecing_path_info.insert(0,component_path,sl,string::npos);
                     component_path.erase(sl);
                 }
                 fi.metavars["SITECING_PATH_INFO"]=sitecing_path_info;
                 fi.metavars["SITECING_COMPONENT_PATH"]=component_path;
                 action = component_path;
                 action_handler = config.lookup_action_handler(component_path);
                 if(action_handler) {
                     action = action_handler->action;
                 }
                 string pwd = konforka::dir_name(full_component_path);
                 if(chdir(pwd.c_str()))
                     throw konforka::exception(CODEPOINT,"failed to chdir() into document's directory");
                 so_component soc = ss.fetch(action,&scif);
                 if(action_handler) {
                     soc.ac->run(__magic_action,
                             config.root_source.c_str(), config.root_intermediate.c_str(), config.root_so.c_str(),
                             &(action_handler->args)
                             );
                 }else{
                     soc.ac->main(0,NULL);
                 }
             }catch(http_status& hs) {
                 scif.headers["Status"] = hs.status+" "+hs.message;
                 string hshp = config.lookup_http_status_handler(component_path,hs.status);
                 if(!hshp.empty()) {
                     so_component hsh = ss.fetch(hshp,&scif); // TODO: handle error trying to handle status
                     hsh.ac->run(__magic_http_status,config.root_source.c_str(),config.root_intermediate.c_str(),
                             config.root_so.c_str(),action.c_str(),&hs);
                 }
             }catch(compile_error& ce) {
                 config_options *co_exception_handler = config.lookup_config(component_path,config_options::flag_exception_handler);
                 if(co_exception_handler) {
                     so_component eh = ss.fetch(co_exception_handler->exception_handler,&scif); // TODO: handle error trying to handle error.
                     eh.ac->run(__magic_compile_error,ce.what(),config.root_source.c_str(),config.root_intermediate.c_str(),config.root_so.c_str(),ce.component_path.c_str());
                 }else{
                     ce.see(CODEPOINT);
                     throw;
                 }
             }catch(preprocessor_error& pe) {