Attachment 221404 Details for Bug 252094 – Avoid side-effects from getgr(nam|gid) on getgrent

[patch] Avoid side-effects from getgr(nam|gid) on getgrent

grp.diff (text/plain), 2.03 KB, created by Viktor Dukhovni on 2021-01-08 23:15:54 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Viktor Dukhovni

Created: 2021-01-08 23:15:54 UTC

Size: 2.03 KB

patch

obsolete

>Index: getgrent.c
>===================================================================
>--- getgrent.c	(revision 367119)
>+++ getgrent.c	(working copy)
>@@ -869,16 +869,17 @@
> 		}
> 		fresh = 1;
> 	}
>-	if (how == nss_lt_all)
>+	if (how == nss_lt_all) {
> 		stayopen = 1;
>-	else {
>-		if (!fresh)
>+		pos = ftello(st->fp);
>+	} else {
>+		if (!fresh) {
>+			pos = ftello(st->fp);
> 			rewind(st->fp);
>+		}
> 		stayopen = st->stayopen;
> 	}
> 	rv = NS_NOTFOUND;
>-	if (stayopen)
>-		pos = ftello(st->fp);
> 	while ((line = fgetln(st->fp, &linesize)) != NULL) {
> 		if (line[linesize-1] == '\n')
> 			linesize--;
>@@ -900,7 +901,7 @@
> 		    &buffer[linesize + 1], bufsize - linesize - 1, errnop);
> 		if (rv & NS_TERMINATE)
> 			break;
>-		if (stayopen)
>+		if (how == nss_lt_all)
> 			pos = ftello(st->fp);
> 	}
> 	if (st->fp != NULL && !stayopen) {
>@@ -907,6 +908,8 @@
> 		fclose(st->fp);
> 		st->fp = NULL;
> 	}
>+	if (st->ftp != NULL && how != nss_lt_all)
>+		fseeko(st->fp, pos, SEEK_SET);
> 	if (rv == NS_SUCCESS && retval != NULL)
> 		*(struct group **)retval = grp;
> 	else if (rv == NS_RETURN && *errnop == ERANGE && st->fp != NULL)
>@@ -1351,11 +1354,14 @@
> 		}
> 		fresh = 1;
> 	}
>-	if (how == nss_lt_all)
>+	if (how == nss_lt_all) {
> 		stayopen = 1;
>-	else {
>-		if (!fresh)
>+		pos = ftello(st->fp);
>+	} else {
>+		if (!fresh) {
>+			pos = ftello(st->fp);
> 			rewind(st->fp);
>+		}
> 		stayopen = st->stayopen;
> 	}
> docompat:
>@@ -1418,8 +1424,6 @@
> 		break;
> 	}
> 	rv = NS_NOTFOUND;
>-	if (stayopen)
>-		pos = ftello(st->fp);
> 	while ((line = fgetln(st->fp, &linesize)) != NULL) {
> 		if (line[linesize-1] == '\n')
> 			linesize--;
>@@ -1460,7 +1464,7 @@
> 		    &buffer[linesize + 1], bufsize - linesize - 1, errnop);
> 		if (rv & NS_TERMINATE)
> 			break;
>-		if (stayopen)
>+		if (how == nss_lt_all)
> 			pos = ftello(st->fp);
> 	}
> fin:
>@@ -1468,6 +1472,8 @@
> 		fclose(st->fp);
> 		st->fp = NULL;
> 	}
>+	if (st->fp != NULL && how != nss_lt_all)
>+		fseeko(st->fp, pos, SEEK_SET);
> 	if (rv == NS_SUCCESS && retval != NULL)
> 		*(struct group **)retval = grp;
> 	else if (rv == NS_RETURN && *errnop == ERANGE && st->fp != NULL)

Actions: View | Diff

Attachments on bug 252094: 220877 | 221404 | 221407 | 221430