Found my mistake: I should have been copying from _nl in the on-disk

case, not nl (size we take its size - d'oh!).
This commit is contained in:
Jordan K. Hubbard 1996-10-06 02:56:22 +00:00
parent e6068a345f
commit ed438602b1
1 changed files with 15 additions and 11 deletions

View File

@ -24,7 +24,7 @@
* THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
* library functions for userconfig library * library functions for userconfig library
* *
* $Id: uc_main.c,v 1.9 1996/10/05 13:30:43 jkh Exp $ * $Id: uc_main.c,v 1.10 1996/10/05 16:33:05 jkh Exp $
*/ */
#include <sys/types.h> #include <sys/types.h>
@ -99,34 +99,38 @@ uc_open(char *name){
free(kern); free(kern);
return NULL; return NULL;
} }
nl = (struct nlist *)malloc((size + 1) * sizeof(struct nlist)); kern->nl = nl = (struct nlist *)malloc((size + 1) * sizeof(struct nlist));
for (i = 0; i < size; i++) { for (i = 0; i < size; i++) {
char name[255]; char *cp, name[255];
int c1; int c1;
unsigned int uc1; unsigned int uc1;
short d1; short d1;
unsigned long v1; unsigned long v1;
if (fscanf(fp, "%s %u %d %hd %ld\n", name, &uc1, &c1, &d1, &v1) == 5) { if (fgets(name, 255, fp) == NULL) {
nl[i].n_name = strdup(name); msgDebug("Can't get name field for entry %d\n", i);
free(kern);
return NULL;
}
if ((cp = index(name, '\n')) != NULL)
*cp = '\0';
nl[i].n_name = strdup(name);
if (fscanf(fp, "%u %d %hd %ld\n", &uc1, &c1, &d1, &v1) == 5) {
nl[i].n_type = (unsigned char)uc1; nl[i].n_type = (unsigned char)uc1;
nl[i].n_other = (char)c1; nl[i].n_other = (char)c1;
nl[i].n_desc = d1; nl[i].n_desc = d1;
nl[i].n_value = v1; nl[i].n_value = v1;
if (isDebug()) if (isDebug())
msgDebug("Entry %d, decoded: \"%s\", %d %d %hd %ld\n", i, nl[i].n_name, nl[i].n_type, nl[i].n_other, nl[i].n_desc, nl[i].n_value); msgDebug("uc_open: for entry %d, decoded: \"%s\", %d %d %hd %ld\n", i, nl[i].n_name, nl[i].n_type, nl[i].n_other, nl[i].n_desc, nl[i].n_value);
} }
else { else {
nl[i].n_name = "";
nl[i].n_type = 0; nl[i].n_type = 0;
nl[i].n_other = 0; nl[i].n_other = 0;
nl[i].n_desc = 0; nl[i].n_desc = 0;
nl[i].n_value = 0; nl[i].n_value = 0;
} }
} }
nl[i].n_name = NULL;
fclose(fp); fclose(fp);
kern->nl = nl;
i = 0; i = 0;
} }
else else
@ -142,8 +146,8 @@ uc_open(char *name){
#else #else
{ {
#endif #endif
kern->nl=(struct nlist *)malloc(sizeof(nl)); kern->nl=(struct nlist *)malloc(sizeof(_nl));
bcopy(nl, kern->nl, sizeof(nl)); bcopy(_nl, kern->nl, sizeof(_nl));
} }
if (incore) { if (incore) {