Кто у нас лихо на С лабает (часть2)

вот мучаю такую конструкцию(кусок плугина к миранде, который позволяет сохранять и поднимать контакт-лист в удобоваримом виде):

void createParentGroups(const char *groupName)
{
	if ( strrchr(groupName,'\\') != NULL )
	{
		char *duplicate;
		duplicate = strdup(groupName);
		if( duplicate == NULL )
			MessageBox(NULL, Translate("Insufficient memory available"), Translate("Error"),MB_OK);
		else
		{
			char *pos;
			pos = strrchr(duplicate,'\\'); //Find the last \ character
			if (pos != NULL)
			{
				*pos = 0; // Nullterminate the string where the last \ was found
				createGroup(duplicate);
			}
			else
			{
				MessageBox(NULL, Translate("A critical error occured.\nTry to do the operation one more time."), Translate("Critical error"),MB_OK);
			}
		}
		free( duplicate ); //Since strdup calls malloc it is good to free the space allocated
	}

	return;
}


void createGroup(const char *groupName)
{
	if (!groupExistsOLD(groupName))
	{
		int groupId;
		char groupIdStr[11];
		DBVARIANT dbv;
		char *groupName2;
		createParentGroups(groupName); // Create the parentgroups if needed
		groupName2 = malloc(strlen(groupName + 2));
		groupName2[0] = 1; // This is because the DB requires the groupname to begin with the second character in the string
		groupName2[1] = 0;
		strcat(groupName2, groupName);
		// Find the right id for the group
		for(groupId = 0;; groupId++)
		{
			itoa(groupId, groupIdStr,10);
			if(DBGetContactSetting(NULL, "CListGroups", groupIdStr, &dbv))
				break;
			DBFreeVariant(&dbv);
		}
		DBWriteContactSettingString(NULL,"CListGroups", groupIdStr, groupName2);
		free( groupName2 );
	}
	return;
}

Вызывается в случае, если при импорте контакта не оказалось группы, в которой он должен лежать.

Группа передается в аргументах в виде Жилкомсбыт\По работе с физическими лицами отдел

... эта конструкция виснет на второй итерации при попытке выполнить free( groupName2 ); ...

... закоментил, откомпилил ... на машине с двумя гигами оперативы пашет замечательно ... на 256 - вылетает после создания первой группы (но первую - создает)

... вижу, что написано умнО ... но коряво с точки зрения использования памяти. ... что делать?

Добавлено спустя 1 час 30 минут 54 секунды:

Вроде нашел косяк ... только не совсем понятно, почему оно начало работать ... поменял

groupName2 = malloc(strlen(groupName + 2)); 

groupName2 = malloc(strlen(groupName) + 2);