strcpy/writeable проблема.

[coNsept]

Вообщем имеется следующий момент кода, изза которого возникает проблема с записью по памяти.

typedef struct SETITEM_OPTIONINFO
{
 char Name[255];	
} 
SETITEM_OPTIONINFO, *LPSETITEM_OPTIONINFO;

class CWzMemScript
{
public:
 char *GetString();

private:
 char TokenString[100];
};

char *CWzMemScript::GetString()
{
 return &this->TokenString[0];
}

То есть все очень просто, TokenString заполняется определенными данными и копируется в Name.
Олька мне сказала, что при попытке копирования strcpy выдает ошибку мол о проблеме с записью по памяти (access writeable)
Естественно секции кода и данных установлены в writeable/readable.

LPSETITEM_OPTIONINFO p;
strcpy(&p->Name[0], WzMemScript.GetString());

wtf? Вроде все правильно.

[pproger]

LPSETITEM_OPTIONINFO p;
strcpy(&p->Name[0], WzMemScript.GetString());

Олька мне сказала, что при попытке копирования strcpy выдает ошибку мол о проблеме с записью по памяти (access writeable)

а чего ты ожидал? память под p не выделена то

про ужасный код я вообще молчу

char *CWzMemScript::GetString()
{
return &this->TokenString[0];
}

char *CWzMemScript::GetString()
{
return this->TokenString;
}

не?
а еще лучше и ПРАВИЛЬНЕЕ

const char *CWzMemScript::GetString() const
{
return this->TokenString;
}

[coNsept]

Код не мной написан, я выложил первоначальный вариант.
На самом деле я память выделял, но все тоже самое.

p = new LPSETITEM_OPTIONINFO;
strcpy(&p->Name[0], WzMemScript.GetString());
delete p;

[pproger]

2coNsept

p = new LPSETITEM_OPTIONINFO;

)))))))))))))))))))) ты это, завязывай с тяжелыми наркотиками. выделял он память. под ЧТО ты ее выделил?

LPSETITEM_OPTIONINFO p = new SETITEM_OPTIONINFO;
strcpy(&p->Name[0], WzMemScript.GetString());
delete p;

[coNsept]

Собственно я так и сделал. Но увы, все равно тоже самое, я озадачен.

const char *CWzMemScript::GetString() const
{
 return this->TokenString;
}

LPSETITEM_OPTIONINFO p = new SETITEM_OPTIONINFO;
strcpy(&p->Name[0], WzMemScript.GetString());
delete p;

[pproger]

2coNsept

LPSETITEM_OPTIONINFO p = new SETITEM_OPTIONINFO;
strcpy(&p->Name[0], WzMemScript.GetString());
delete p;

абсолютно валидный код. может у тебя WzMemScript.GetString() невалидную строку возвращает? или возвращает строку длиннее Name?

LPSETITEM_OPTIONINFO p = new SETITEM_OPTIONINFO;
strcpy(&p->Name[0], "some_str");
delete p;

попробуй

[coNsept]

Мдя, все равно. Что-то мне кажется, что тут &p->Name[0] виновник, хотя вроде все нормально.
Функция копирует первые 4 символа, дальше говорит о writeable.

[pproger]

2coNsept
создай пустой проект, помести туда определение и синонимы своей структуры, и в main вставь ИМЕННО ЭТОТ код

LPSETITEM_OPTIONINFO p = new SETITEM_OPTIONINFO;
strcpy(&p->Name[0], "some_str");
delete p;

ты что-то делаешь невнимательно

[coNsept]

#include "MyApp.h"
#include <stdio.h>
#include <conio.h>
#include <string.h>
#include <Windows.h>

typedef struct SETITEM_OPTIONINFO	// 50
{
	char Name[255];	
} 
SETITEM_OPTIONINFO, *LPSETITEM_OPTIONINFO;

class CWzMemScript
{
public:
	CWzMemScript() {};
	virtual ~CWzMemScript() {};
	const char *GetString() const;
 void SetString(const char Str[]);

private:
	char TokenString[100];
};

const char *CWzMemScript::GetString() const
{
	return this->TokenString;
}

void CWzMemScript::SetString(const char Str[]) 
{
	strcpy(this->TokenString, Str); 
}

int main(void)
{
	CWzMemScript WzMemScript;
	LPSETITEM_OPTIONINFO p = new SETITEM_OPTIONINFO;
	WzMemScript.SetString("Hello World");
	strcpy(p->Name, WzMemScript.GetString());
	puts(p->Name);
	_getch();
	return 0;
}

Вообщем работает все нормально, но в проекте почему-то не хочет... Ладно pproger спасибо тебе за помощь, буду копать.