@@ -19,7 +19,7 @@ function request(querytext, func, errorfunc) {

 donewpost.onclick = function(e) {

 	request("/newpost"+window.location.search+"&t="+encodeURIComponent(document.getElementById("wysihtml5-editor").value),function(r) {

-		window.location="/posts.html"+window.location.search+"&n=r";},function() {});

+		window.location="/posts.html"+window.location.search;},function() {});

 };

 }());

@@ -24,11 +24,11 @@ gotonewpost.onclick = function(e) {

 document.getElementById("posts").innerHTML='Loading posts...';

 request("/lastpost"+window.location.search,function(r) {

-	request("/getpost"+window.location.search+"n="+r, function(r1) {

+	request("/getpost"+window.location.search+"&n="+r, function(r1) {

 		document.getElementById("posts").innerHTML=r1;

 	},function() {

 		document.getElementById("posts").innerHTML='Error getting posts';

-	}

+	});

 },function() {

 	document.getElementById("posts").innerHTML='There are no posts';

 });

@@ -334,7 +334,7 @@ http_lastpost(wk *web, int connid, wk_uri *uri, void *userptr,char *user)

         char reply[64];

         int n;

         kakumei *ka=(kakumei *)userptr;

-        if((n=post_last(ka))<=0) {

+        if((n=post_last(ka))<0) {

                 wk_serve_error(web,connid,wkerr_internal);

                 log_write("LSTP","invalid postnum, send error");

                 return(wkact_finished);

@@ -15,6 +15,7 @@

 #include <sys/types.h>

 #include <sys/stat.h>

 #include <time.h>

+#include "loglib.h"

 #include "kakumei.h"

 #include "kakumei_posts.h"

@@ -29,8 +30,10 @@ post_new(kakumei *ka, char *user, char *text)

         char dirname[1024];

         FILE *f;

         struct stat st;

-        if(ka==NULL || user==NULL || user[0]=='\0' || text==NULL || text[0]=='\0')

+        if(ka==NULL || user==NULL || user[0]=='\0' || text==NULL || text[0]=='\0') {

+		log_write("POST","sanity check error\n");

                 return(-1);

+	}

         last=post_last(ka);

         filename[0]='\0';

         for(num=last+1;num<MAXPOSTS;num++) {

@@ -39,8 +42,10 @@ post_new(kakumei *ka, char *user, char *text)

                 if(stat(filename,&st)!=0)

                         break; /* this one is ok: file doesn't exist */

         }

-        if(filename[0]=='\0' || num>=MAXPOSTS)

+        if(filename[0]=='\0' || num>=MAXPOSTS) {

+		log_write("POST","couldn't generate filename or too many posts\n");

                 return(-1);

+	}

         /* create dir structure */

         mkdir(DATADIR,0700);

         mkdir(POSTSDIR,0700);

@@ -48,12 +53,13 @@ post_new(kakumei *ka, char *user, char *text)

         dirname[sizeof(dirname)-1]='\0';

         mkdir(dirname,0700);

         /* save the new post */

-        if((f=fopen(filename,"w"))==NULL)

+        if((f=fopen(filename,"w"))==NULL) {

+		log_write("POST","couldn't save post\n");

                 return(-1);

+	}

         fprintf(f,"{\n  \"author\":\"%s\",\n",user);

         fprintf(f,"  \"date\":%li,\n",(long)time(NULL));

         fprintf(f,"  \"text\":\"%s\",\n",text);

-        fprintf(f,"  \"text\":\"%s\",\n",text);

         fprintf(f,"  \"comments\":[\n]}\n");

         fclose(f),f=NULL;

         /* save the "last" mark */

@@ -74,8 +80,10 @@ post_last(kakumei *ka)

         int fd;

         snprintf(filename,sizeof(filename)-1,"%s/%s",POSTSDIR,"last");

         filename[sizeof(filename)-1]='\0';

-        if((fd=open(filename,O_RDONLY))==-1)

+        if((fd=open(filename,O_RDONLY))==-1) {

+		log_write("POST","couldn't get last post number\n");

                 return(-1);

+	}

         memset(readbuf,0,sizeof(readbuf));

         read(fd,readbuf,sizeof(readbuf)-1);

         close(fd),fd=-1;

@@ -86,12 +94,16 @@ int

 post_get(kakumei *ka, int num, char *filename, int filenamesize)

 {

         struct stat st;

-        if(ka==NULL || num<0 || filename==NULL || filenamesize<=0)

+        if(ka==NULL || num<0 || filename==NULL || filenamesize<=0) {

+		log_write("POST","Sanity check error\n");

                 return(-1);

+	}

         snprintf(filename,filenamesize,"%s/%04i/%04i.json",POSTSDIR,num/10000,num%10000);

         filename[filenamesize-1]='\0';

-        if(stat(filename,&st)!=0)

+        if(stat(filename,&st)!=0) {

+		log_write("POST","Post file doesn't exist\n");

                 return(-1); /* file doesn't exist */

+	}

         return(0);

 }

@@ -101,15 +113,20 @@ post_addcomment(kakumei *ka, int postnum, char *user, char *text)

         char filename[1024];

         char buf[16];

         FILE *f;

-        if(ka==NULL || postnum<0 || user==NULL || text==NULL)

+        if(ka==NULL || postnum<0 || user==NULL || text==NULL) {

+		log_write("POST","Sanity check error\n");

                 return(-1);

-        if(post_get(ka,postnum,filename,sizeof(filename))!=0)

+	}

+        if(post_get(ka,postnum,filename,sizeof(filename))!=0) 

                 return(-1);

-        if((f=fopen(filename,"r+"))==NULL)

+        if((f=fopen(filename,"r+"))==NULL) {

+		log_write("POST","Couldn't open post file fot appending\n");

                 return(-1);

+	}

         fseek(f,-2,SEEK_END);

         if(fread(buf,1,2,f)!=2 || buf[0]!=']' ||  buf[1]!='}') {

                 fclose(f),f=NULL;

+		log_write("POST","Post file contents are corrupted\n");

                 return(-1);

         }

         fseek(f,-2,SEEK_END);