don't use unbounded operations

This commit is contained in:
Hisham Muhammad 2009-06-02 04:49:26 +00:00
parent b62f9cea6f
commit 1371ee28a7
1 changed files with 4 additions and 4 deletions

View File

@ -86,7 +86,7 @@ static unsigned long int parseBatInfo(const char *fileName, const unsigned short
const FILE *file; const FILE *file;
char line[50]; char line[50];
sprintf((char *) infoPath, "%s%s/%s", batteryPath, newEntry->content, fileName); snprintf((char *) infoPath, sizeof infoPath, "%s%s/%s", batteryPath, newEntry->content, fileName);
if ((file = fopen(infoPath, "r")) == NULL) { if ((file = fopen(infoPath, "r")) == NULL) {
return 0; return 0;
@ -135,7 +135,7 @@ static ACPresence chkIsOnline() {
char statePath[50]; char statePath[50];
sprintf((char *) statePath, "%s/%s/state", power_supplyPath, entryName); snprintf((char *) statePath, sizeof statePath, "%s/%s/state", power_supplyPath, entryName);
file = fopen(statePath, "r"); file = fopen(statePath, "r");
if (!file) { if (!file) {
@ -190,7 +190,7 @@ static ACPresence chkIsOnline() {
} }
char onlinePath[50]; char onlinePath[50];
sprintf((char *) onlinePath, "%s/%s/online", power_supplyPath, entryName); snprintf((char *) onlinePath, sizeof onlinePath, "%s/%s/online", power_supplyPath, entryName);
file = fopen(onlinePath, "r"); file = fopen(onlinePath, "r");
if (!file) { if (!file) {
@ -264,7 +264,7 @@ static double getSysBatData() {
const char ueventPath[50]; const char ueventPath[50];
sprintf((char *) ueventPath, "%s%s/uevent", power_supplyPath, entryName); snprintf((char *) ueventPath, sizeof ueventPath, "%s%s/uevent", power_supplyPath, entryName);
FILE *file; FILE *file;
if ((file = fopen(ueventPath, "r")) == NULL) { if ((file = fopen(ueventPath, "r")) == NULL) {