file is not of required architecture warning

[agentsmith]

here is my hash table code that reads in a file and puts it into a hash table and handles collisions with a linked list approach.

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

typedef struct List { char *word; struct List *next; } List;

#define HASHSIZE 65536
#define LEN 50

static struct List *hashtab[HASHSIZE];

static FILE *open_file (char *file, char *mode)
{
  FILE *fp = fopen (file, mode);

  if (fp == NULL){
    perror ("Unable to open file");
    exit (EXIT_FAILURE);
  }
	return fp;
}

List *list_alloc (void)
{
	List *tmp = calloc (1, sizeof (List));
		if (tmp == NULL){
			fprintf (stderr, "Error: calloc ()\n");
			}
			tmp -> word = NULL;
			tmp -> next = NULL;
			return tmp;
}


int is_punctuation (int a)
{
    return a == ' ' || a == '\n' ||
           a == EOF || a == ',' || 
           a ==';' || a =='.'||
           a =='!' || a =='?' || 
           a ==':' || a =='`';
}


unsigned rot_hash (void *key) 
{
	unsigned char *p = key;
	unsigned h = 0;
	int i;
	
	for (i = 0; i < strlen(key); i++)
		h = (h << 4) ^ (h >> 28) ^ p[i];
	return h % HASHSIZE;
}

void install (char *s)
{
	List *tmp = NULL;
	unsigned int h = rot_hash (s);
	if (hashtab[h] -> word == NULL){
		hashtab[h] = list_alloc ();
		strcpy (hashtab[h] -> word, s);
		printf ("Put value into empty space\n");
	}else{
		for (tmp = hashtab[h]; tmp != NULL; tmp = tmp -> next){	
			if (tmp -> next == NULL){
				tmp -> next = list_alloc ();
				strcpy (tmp -> next -> word, s);
				printf ("Put value into a linked list\n");
				break;
			}
		}
	}
}
				

int main (int argc, char *argv[])
{
	int x;
	char a, word[LEN];
	x = 0;
	FILE *fp = open_file (argv[1], "r");
	do{
		a = fgetc (fp);
		if (is_punctuation(a)){
			if (x != 0){
				word[x] = '\0';
				install (word);
				x = 0;
				}
		}else{
			word[x] = tolower(a);
			x++;
			}
	} while (a != EOF);
	fclose (fp);
	return 0;
}

for some reason i keep getting a bus error and a warning that says the file is not of required architecture but i don't see any mistakes in the code. where am i going wrong?
thanks

[Salem]

Which OS/Compiler are you talking about?

The exact text of the error message(s), and what you did to compile the code would help.

[agentsmith]

I am compiling on Mac OS X 10.5 using GCC compiler.
this in the command i use: gcc hcopy.c -Wall hcopy
and this is the warning: ld: warning in hcopy, file is not of required architecture
thanks for help

[Salem]

Maybe
gcc -o hcopy -Wall hcopy.c

Without the -o, gcc will treat your resulting executable as an input file, not an output file.

[agentsmith]

thanks. it doesn't give the warning anymore but when i run the program it still says bus error. I don't see any obvious mistakes in the code, so where could i be going wrong?

[Salem]

Well one cause would be not passing a command line parameter.
Which you don't check to see if it exists, nor do you check to see if the file was opened successfully.

[agentsmith]

nope i didnt forget to pass the command line parameter, could it be something to do with me using calloc in list_alloc as opposed to malloc?

[Salem]

You're not allocating any space for ->word either, before you do a copy.

[agentsmith]

i thought calling list_alloc before i hash the word would allocate the space required. I am sorry if i am being a little slow, but still don't understand where my mistake is.
thanks

[Salem]

p->word = malloc( strlen(s) + 1 );
strcpy( p->word, s );