[c]funzione penosa da sistemare

[NA01]

salve gente!
ho scritto una bella funzioncina come parte di un progetto che non fa altro che prendere il contenuto di file simili a quello sotto e restituire uno alla volta i percorsi delle librerie condivise (quelle che sulla seconda colonna hanno una x)

08048000-0804f000 r-xp 00000000 03:41 179545     /usr/sbin/cron
0804f000-08050000 rw-p 00007000 03:41 179545     /usr/sbin/cron
08050000-08071000 rw-p 08050000 00:00 0          [heap]
49b26000-49b36000 r-xp 00000000 03:41 167470     /lib/libselinux.so.1
49b36000-49b37000 rw-p 00010000 03:41 167470     /lib/libselinux.so.1
4a19c000-4a1ae000 r-xp 00000000 03:41 179414     /lib/tls/libnsl-2.3.5.so
4a1ae000-4a1b0000 rw-p 00011000 03:41 179414     /lib/tls/libnsl-2.3.5.so
4a1b0000-4a1b2000 rw-p 4a1b0000 00:00 0 
4e617000-4e62c000 r-xp 00000000 03:41 20093      /lib/ld-2.3.5.so
4e62c000-4e62e000 rw-p 00014000 03:41 20093      /lib/ld-2.3.5.so
4e742000-4e873000 r-xp 00000000 03:41 166233     /lib/tls/libc-2.3.5.so
4e873000-4e874000 r--p 00130000 03:41 166233     /lib/tls/libc-2.3.5.so
4e874000-4e877000 rw-p 00131000 03:41 166233     /lib/tls/libc-2.3.5.so
4e877000-4e87a000 rw-p 4e877000 00:00 0 
4e904000-4e906000 r-xp 00000000 03:41 173226     /lib/tls/libdl-2.3.5.so
4e906000-4e908000 rw-p 00001000 03:41 173226     /lib/tls/libdl-2.3.5.so
b7ea9000-b7eb3000 r-xp 00000000 03:41 51412      /lib/tls/libnss_files-2.3.5.so
b7eb3000-b7eb5000 rw-p 00009000 03:41 51412      /lib/tls/libnss_files-2.3.5.so
b7eb5000-b7ebc000 r-xp 00000000 03:41 51410      /lib/tls/libnss_compat-2.3.5.so
b7ebc000-b7ebe000 rw-p 00006000 03:41 51410      /lib/tls/libnss_compat-2.3.5.so
b7ef5000-b7f16000 rw-p b7ef5000 00:00 0 
b7f16000-b7f1d000 r-xp 00000000 03:41 2354       /lib/libpam.so.0.79
b7f1d000-b7f1e000 rw-p 00007000 03:41 2354       /lib/libpam.so.0.79
b7f29000-b7f32000 r-xp 00000000 03:41 51414      /lib/tls/libnss_nis-2.3.5.so
b7f32000-b7f34000 rw-p 00008000 03:41 51414      /lib/tls/libnss_nis-2.3.5.so
b7f34000-b7f36000 rw-p b7f34000 00:00 0 
bfe21000-bfe36000 rw-p bfe21000 00:00 0          [stack]
ffffe000-fffff000 ---p 00000000 00:00 0          [vdso]

ora come ora ho una funzione che fa correttamente il lavoro, ma è scritta in modo penoso.
ve la propongo:

char * getNextLibrariePath(FILE **file){
  static char buffer[MAX_LINE];
  char *temp = NULL;
  size_t n;
  FILE *proc = *file;
  while(1){
    if((fscanf(proc,"%s",buffer)) != 1){
      break;
    }
    if((fscanf(proc,"%s",buffer)) != 1){
      break;
    }
    if(strstr(buffer,"x") == NULL){
      getline(&temp,&n,proc);
      continue;
    }
    if((fscanf(proc,"%s",buffer)) != 1){
      break;
    }
    if((fscanf(proc,"%s",buffer)) != 1){
      break;
    }
    if((fscanf(proc,"%s",buffer)) != 1){
      break;
    }
    if((fscanf(proc,"%s",buffer)) != 1){
      break;
    }
    return buffer;
  }
  return NULL;
}

a parte la serie di if e di break/continue che vorrei evitare se si potesse ho dei seri dubbi sull'uso di buffer.
non so quanto sia il limite di lunghezza, e questo mi creerebbe un problema di sicurezza in un demone che deve funzionare da firewall.

idee su come risolvere che non comportino il passaggio al c++ (ho già pensato al port che mi darebbe molti vantaggi anche in altri punti, ma sarebbe tanto lavoro che ritarderebbe ancora l'uscita di una prima versioen anche solo semi funzionante).

ciao!

[ilsensine]

Un buon esempio vale più di mille parole...

Compila e prova questo programma; per ogni dubbio, chiedi a man fscanf

#include <stdio.h>
#include <stdlib.h>

int main(void)
{
	FILE *f = fopen("/proc/self/maps", "rb");
	char *buf;
	long long unsigned ino;
	char type[5];

	while (fscanf(f, "%*s%4s%*s%*s%lld", type, &ino)==2) {
		if (ino && fscanf(f, "%as", &buf)==1) {
			if (type[2]=='x' && ino)
				printf("Happy hacking on file %s\n", buf);
			free(buf);
		}
	}

	fclose(f);
	return 0;
}

[ilsensine]

oops meglio questo (perché? )

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

int main(void)
{
	FILE *f = fopen("/proc/self/maps", "rb");
	char buf[4096];
	long long unsigned ino;
	char type[5];

	while (fscanf(f, "%*p-%*p%04s%*s%*s%lld", type, &ino)==2) {
		if (ino && fgets(buf, 4096, f)) {
			char *p = strchr(buf, '\n');
			if (!p) {
				printf("mmm strange file name...\n");
				exit(-1);
			} else {
				if (type[2]=='x' && ino) {
					*p = '\0';
					p = strchr(buf, '/');
					printf("Happy hacking on file %s\n", p);
				}
			}
		}
	}

	fclose(f);
	return 0;
}