From f7081b77b0efb1e0cd8dc6a65664c37f004643cb Mon Sep 17 00:00:00 2001
From: deuce <>
Date: Mon, 8 May 2006 19:37:19 +0000
Subject: [PATCH] Use alloca() instead of malloc()/free() where possible.
---
src/sbbs3/jsexec.c | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/src/sbbs3/jsexec.c b/src/sbbs3/jsexec.c
index 929779e93d..ce6a8e2221 100644
--- a/src/sbbs3/jsexec.c
+++ b/src/sbbs3/jsexec.c
@@ -280,7 +280,7 @@ js_read(JSContext *cx, JSObject *obj, uintN argc, jsval *argv, jsval *rval)
if(argc)
JS_ValueToInt32(cx,argv[0],&len);
- if((buf=malloc(len))==NULL)
+ if((buf=alloca(len))==NULL)
return(JS_TRUE);
rd=fread(buf,sizeof(char),len,stdin);
@@ -288,7 +288,6 @@ js_read(JSContext *cx, JSObject *obj, uintN argc, jsval *argv, jsval *rval)
if(rd>=0)
*rval = STRING_TO_JSVAL(JS_NewStringCopyN(cx,buf,rd));
- free(buf);
return(JS_TRUE);
}
@@ -301,7 +300,7 @@ js_readln(JSContext *cx, JSObject *obj, uintN argc, jsval *argv, jsval *rval)
if(argc)
JS_ValueToInt32(cx,argv[0],&len);
- if((buf=malloc(len+1))==NULL)
+ if((buf=alloca(len+1))==NULL)
return(JS_TRUE);
p=fgets(buf,len+1,stdin);
@@ -309,7 +308,6 @@ js_readln(JSContext *cx, JSObject *obj, uintN argc, jsval *argv, jsval *rval)
if(p!=NULL)
*rval = STRING_TO_JSVAL(JS_NewStringCopyZ(cx,truncnl(p)));
- free(buf);
return(JS_TRUE);
}
--
GitLab