Access violation writing location

Reply to Access violation writing location on Thu, 10 Jun 2010 15:47:43 GMT

jonnybx — Thu, 10 Jun 2010 15:47:43 GMT

Mal ne frage:
was hat "Access violation writing location" als debug-fehlermeldung zu bedeuten?
Kann mir jemand helfen?

Reply to Access violation writing location on Thu, 10 Jun 2010 15:57:53 GMT

Janjan — Thu, 10 Jun 2010 15:57:53 GMT

Du greifst auf Speicher zu, der dir nicht gehört.
Irgendwo machst du deine Zeiger kaputt.

Reply to Access violation writing location on Thu, 10 Jun 2010 15:58:32 GMT

Nukularfüsiker — Thu, 10 Jun 2010 15:58:32 GMT

Das ist eine Schutzverletzung. Du greifst auf Speicheradressen zu, auf die dein Programm nicht zugreifen darf. Das sind häufig Null-Pointer-Dereferenzierungen, oder Zugriffe auf bereits freigegebenen Speicher.

Reply to Access violation writing location on Thu, 10 Jun 2010 16:24:32 GMT

jonnybx — Thu, 10 Jun 2010 16:24:32 GMT

ok, aber wo zmach ich hier die zeiger kaputt?

Main.cpp:

#include 
#include "Keyings.h"
using namespace std;
using namespace keys;

int main (void)
{
	char teststring;
	cout << "press any key to start: \n(if you don't find any use j)\n";
	cin >> teststring;
	picturergb* pic1 = new picturergb;
	picturergb* pic2 = new picturergb;
	picturergb* pic3 = new picturergb;
	textkey* txt = new textkey;
	cout << "\ncreated two imagematrixes\n";
	cout << sizeof(picturergb) << "\n";
#ifdef _OPENMP
	cout << "found OpenMP, dividing into 4 threads\n";
	for(int i = 0; i<1500;i++)//schleife 1
	{
#pragma omp parallel sections
		{

#pragma omp section
			{
				int fader = i;
				int faderinv = 128-fader;
				for(int g=0;g<320;g++)
				{
					int b = g;
					for(int h=0;h<240;h++)
					{
						int l = h;
						pic3->red[l][b] = pic1->red[l][b]/128*fader + pic2->red[l][b]/128*faderinv;
						pic3->blue[l][b] = pic1->blue[l][b]/128*fader + pic2->blue[l][b]/128*faderinv;
						pic3->green[l][b] = pic1->green[l][b]/128*fader + pic2->green[l][b]/128*faderinv;
					}
				}
				cout << "quarter 1 done\n";
			}
#pragma omp section
			{
				int fader = i;
				int faderinv = 128-fader;
				for(int g=320;g<640;g++)
				{
					int b = g;
					for(int h=0;h<240;h++)
					{
						int l = h;
						pic3->red[l][b] = pic1->red[l][b]/128*fader + pic2->red[l][b]/128*faderinv;
						pic3->blue[l][b] = pic1->blue[l][b]/128*fader + pic2->blue[l][b]/128*faderinv;
						pic3->green[l][b] = pic1->green[l][b]/128*fader + pic2->green[l][b]/128*faderinv;
					}
				}
				cout << "quarter 2 done\n";
			}
#pragma omp section
			{
				int fader = i;
				int faderinv = 128-fader;
				for(int g=0;g<320;g++)
				{
					int b = g;
					for(int h=240;h<480;h++)
					{
						int l = h;
						pic3->red[l][b] = pic1->red[l][b]/128*fader + pic2->red[l][b]/128*faderinv;
						pic3->blue[l][b] = pic1->blue[l][b]/128*fader + pic2->blue[l][b]/128*faderinv;
						pic3->green[l][b] = pic1->green[l][b]/128*fader + pic2->green[l][b]/128*faderinv;
					}
				}
				cout << "quarter 3 done\n";
			}
#pragma omp section
			{
				int fader = i;
				int faderinv = 128-fader;
				for(int g=320;g<640;g++)
				{
					int b = g;
					for(int h=240;h<480;h++)
					{
						int l = h;
						pic3->red[l][b] = pic1->red[l][b]/128*fader + pic2->red[l][b]/128*faderinv;
						pic3->blue[l][b] = pic1->blue[l][b]/128*fader + pic2->blue[l][b]/128*faderinv;
						pic3->green[l][b] = pic1->green[l][b]/128*fader + pic2->green[l][b]/128*faderinv;
					}
				}
				cout << "quarter 4 done\n";
			}
		}
	cout << "picture ready!\n";
	pic3 = keys::Keying(pic3,txt); //Hier soll dann, nachdem das bild aus zwei bildern zusammengeflickt worden ist, gekeyt werden.
	}
#else
	for(int i=0;i<1500;i++)
			{
				int fader = i;
				int faderinv = 128-fader;
				for(int g=0;g<640;g++)
				{
					int b = g;
					for(int h=0;h<480;h++)
					{
						int l = h;
						pic3->red[l][b] = pic1->red[l][b]/128*fader + pic2->red[l][b]/128*faderinv;
						pic3->blue[l][b] = pic1->blue[l][b]/128*fader + pic2->blue[l][b]/128*faderinv;
						pic3->green[l][b] = pic1->green[l][b]/128*fader + pic2->green[l][b]/128*faderinv;
					}
				}
				cout << "picture ready\n";
			}
#endif
	cout << "DONE\n";
	delete pic1;
	delete pic2;
	delete pic3;
	delete txt;
	cout << "pic 1 und pic 2 geloescht\n";
	cin >> teststring;
	return 0;
}

Keyings.h

#include "Picture.h"
namespace keys
{
	struct textkey
{
	int key[640][480];
	unsigned int text_color_rgb[3];
};
picturergb* Keying(picturergb* pic = new picturergb,textkey* text = new textkey)
{
	//picturergb* pic = new picturergb;
	for(int g=0;g<640;g++)
	{
		int b = g;
		for(int h=0;h<480;h++)
		{
			int l = h;
			if(text->key[l][b] == 1) // position 2
			{
				pic ->blue[l][b] = text->text_color_rgb[3];
				pic ->red[l][b] = text->text_color_rgb[1];
				pic ->green[l][b] = text->text_color_rgb[2];
			}
			else
			{

			}
		}
	}
	delete text;
	return pic;
	delete pic;
}
};

Picture.h:

struct picturergb
{
	unsigned int red[640][480];
	unsigned int green[640][480];
	unsigned int blue[640][480];
};

ach ja, ich weiß, stil und organisation sind nicht so gut, wäre aber trotzdem nett wenn mir jeemand helfen könnte.

Reply to Access violation writing location on Thu, 10 Jun 2010 16:22:12 GMT

theta — Thu, 10 Jun 2010 16:22:12 GMT

Was verwendest Du für einen Debugger?
Ein vernünftiger Debugger zeugt an wo eine Access Violation passiert.

Simon

Reply to Access violation writing location on Thu, 10 Jun 2010 16:25:13 GMT

jonnybx — Thu, 10 Jun 2010 16:25:13 GMT

ich hab vs2010 proffesional, die exeption passier beim zweiten schleifendurchlauf der schleife bei 1, an der position2 in Keyings.h
hier die komplette fehlermeldung: Unhandled exception at 0x012b1593 in speedtest_001.exe: 0xC0000005: Access violation reading location 0x00d80040.

Reply to Access violation writing location on Thu, 10 Jun 2010 16:34:00 GMT

Nukularfüsiker — Thu, 10 Jun 2010 16:34:00 GMT

Sollen wir jetzt für dich debuggen? Der Debugger ist doch nicht so schwer zu bedienen, wo liegt das Problem? Im Zweifelsfall halt mal durchsteppen, dann sieht man meistens schnell wo es hakt.

ach ja, ich weiß, stil und organisation sind nicht so gut

Ja, aber OpenMP macht ja die dümmsten Fehler wieder wett

Reply to Access violation writing location on Thu, 10 Jun 2010 18:25:10 GMT

Nexus — Thu, 10 Jun 2010 18:25:10 GMT

jonnybx schrieb:

ach ja, ich weiß, stil und organisation sind nicht so gut

Abläufe, die sich sehr ähnlich sehen (und wahrscheinlich mit Copy&Paste dupliziert worden sind), schreien ja geradezu danach, in eine Funktion ausgelagert zu werden. Damit könntest du schon einiges übersichtlicher und damit auch wartbarer und weniger fehleranfällig gestalten.

Reply to Access violation writing location on Thu, 10 Jun 2010 18:38:06 GMT

BasicMan01 — Thu, 10 Jun 2010 18:38:06 GMT

Ich will ja nichts bemängeln aber:

int key[640][480];

//picturergb* pic = new picturergb;
for(int g=0;g<640;g++)
{
    int b = g;
    for(int h=0;h<480;h++)
    {
        int l = h;
        if(text->key[l][b] == 1) // position 2

// >> 1. text->key[h][g] weil h = l und g = b
// >> 2. text->key[479][639] weil h geht bis 479 und g geht bis 639

// >> int key[640][480]; << das ist deine Definition

Fällt dir was auf?

Reply to Access violation writing location on Thu, 10 Jun 2010 20:03:07 GMT

jonnybx — Thu, 10 Jun 2010 20:03:07 GMT

jo, kleiner leichtsinnsfehler am rande xD
ach ja, ich hab das problem gelöst