C / C++ / MFC

15-Nov-09 14:01

what is wrong with my code it's kicking me in the head right now and if a kind soul could help me out i would be very grateful, thanks.

here is the goal:

Use a one dimensional array to solve the following problem. Read in 20 numbers,
each of which is between 10 and 100, inclusive. As each number is read, print it only if it
is not a duplicate of a number already read. Provide for the “worst case” in which all 20
numbers are different. Use the smallest possible array to solve this problem.Well,

#include <iostream>
#include <iomanip>
#include <ctime>
#include <cstdlib>
#include <limits>
const int x = 20;
using namespace std;


int main()

{ 
	
                  int i = 0;
                  int toss = 0;
	              int count=0;
                  int population[20];
                  int num=0;
	              bool duplicate = false;
                  
 
		      while(1)
              { 
                  			   
	              for (int j = 0; j < x; j++)    // test for duplicate random num
	              {
                     
                      srand(time(NULL));
                      num = rand()%90+11;
                      
                    if (num == population[j])
			    
                    {
					duplicate = true;
			        toss++;
				    //break; 
                    }  // exit the loop if true 
		        
                
                    if (!duplicate)
                    {   // if not duplicate
			          
                     population[j] = num;  // fill the dynamic array
                     count++;
                     } 
                  }

			   
                   
                   for (int ii = 0; ii <= count; ii++)
                   {
                   cout << population[ii]<<" ";
                   }
                   cin.clear();
                   cin.get();
                   }
                   }

C^hris L^osinger15-Nov-09 15:16

15-Nov-09 15:16

i think you need to re-think the problem.

here's a hint: the population array is going to start out with 0 items in it. after the first number read, it will have one item in it. after the second, it can have one or two. after the third, one, two or three. you need to keep track.

here's another hint: you're going to have two loops: one going from 0-19, to read the numbers, and another, inside the first, going from 0-N, where N is the number of items you've read and stored.

Omegaclass19-Nov-09 20:56

19-Nov-09 20:56

I turned in my assignment, but it wasn't right, so i lost some points, i was wondering if you could give an example on how to do this the right way? i apologize for asking, but my professor is very hard to understand due to the fact he is not a native speaker of English so mush of what i do know is self taught.

C^hris L^osinger20-Nov-09 4:32

20-Nov-09 4:32

roughly, and in pseudo-code:

int storage[20]
int numStored = 0

// loop over the 20 numbers you need to read
loop 0 to 20, i

   // read a value
   int newVal = read_a_value;

   // see if you've already read that value
   bFound = false

   // loop over stored numbers
   loop 0 to numStored
      // if it's in the storage array, set the bFound flag
      if storage[i] = newVal 
         bFound = true
         break
   .

   // if you didn't find it, store it
   if not bFound and numStored < 20
      storage[numStored] = newVal
      numStored = numStored + 1
.

Omegaclass20-Nov-09 14:56

20-Nov-09 14:56

Thanks Chris for taking your valuable time and helping me out, i really want to learn c++ and plan to take OOP next semester. so far my grades are in the low 90's and don't want to blow it towards the end of this semester.

dose all this belong in the same for loop? i am not sure what you are 
saying in "read a value" could this be what you are 
saying, "newVal = read_a_value[i];" or is 
this it "newVal = rand()%91+10;" 


how do i implement "see if you've already read that number" if after the 
2nd number i will need to see if the previous number has been read and 
chick if it's a duplicate. and each time a new number is generated it 
must be checked to the others to see if its already there. i am having 
a hard time visualizing the logic and i really do appreciate your time 
to help me with this.  



for loop 0 to 20, i
{

   // read a value
   int newVal = read_a_value;

   // see if you've already read that value
   bFound = false

// loop over stored numbers
   loop 0 to numStored
      // if it's in the storage array, set the bFound flag
      if storage[i] = newVal 
         bFound = true
         break

}

C^hris L^osinger20-Nov-09 15:16

20-Nov-09 15:16

Omegaclass wrote:
dose all this belong in the same for loop?

you'll have two loops. one inside the other.

Omegaclass wrote:
i am not sure what you are saying in "read a value"

that's where you get the next input. your original msg says "Read in 20 numbers,
each of which is between 10 and 100, inclusive." so, if the test data is coming from a file, you'll need to open that file then read each number one at a time. or, if it's coming from the keyboard, or from stdin, or cin, etc..

rand() will work for testing, i guess. but that seems like a strange assignment. i know when i would get assignment like this, the instructor would have a set of data files that he'd send into the program, to make sure the app was doing what it was supposed to on known input.

Omegaclass wrote:
how do i implement "see if you've already read that number" if after the 2nd number i will need to see if the previous number has been read and chick if it's a duplicate.

the previous number will be in the storage array - if it was duplicate, it was already there; if it was a new one, then you will have added it. so, you just need to loop through the items in the storage array and compare each one to the new value.

Omegaclass21-Nov-09 20:31

Hooking CreateRemoteThread [modified]

21-Nov-09 20:31

Thank you Chris i get it now. i appreciate your time. this road i am on learning C++ has not been easy, but it is interesting.

hxhl9515-Nov-09 13:50

15-Nov-09 13:50

I seem to have an abundancy of questions at this time of year. Laugh | :laugh:

Anyways, I have a CreateRemoteThread hook that refuses to work. Here's the code:

HANDLE WINAPI MyCreateRemoteThread(HANDLE hProcess, 
				LPSECURITY_ATTRIBUTES lpThreadAttributes, 
				SIZE_T dwStackSize, 
				LPTHREAD_START_ROUTINE lpStartAddress, 
				LPVOID lpParameter, 
				DWORD dwCreationFlags, 
				LPDWORD lpThreadId)
{
	if (GetpIDFromHandle(hProcess)==SelfpID){
		SetLastError(ERROR_ACCESS_DENIED);
		return NULL;
	}
	else {
	/* Call real CreateRemoteThread and return normally */ }

This format seems to work for every other function I've thought of so far (the ones in KERNEL32.DLL, at least), but it doesn't work for CreateRemoteThread. Sure, the hook is reached and if another process is trying to create a thread in my process's context the SetLastError is executed, but then the calling process crashes. I thought it was a problem with the calling process at first (I wrote one just to test), so I took another program that uses CreateRemoteThread and it crashed too.

Is there anything wrong with my code? Thanks in advance.

EDIT: Clarification - GetpIDFromHandle returns the process ID given a process handle (this works perfectly fine) and SelfpID is a global variable containing the self process ID.

modified on Sunday, November 15, 2009 11:34 PM

Naveen15-Nov-09 16:56

15-Nov-09 16:56

hxhl95 wrote:
ure, the hook is reached and if another process is trying to create a thread in my process's context

How is that possible? If you are installing a hook for this api in your process, the hook function will only be called if you call CreateRemoteThread() from your process. If some other process calls CreateRemoteThread to your process context, there is no way the above API to execute...isnt it?

nave

[My Articles] [My Blog]

hxhl9515-Nov-09 17:09

15-Nov-09 17:09

IAT hooking. And unless I'm very much mistaken, it's working. Smile | :)

Naveen15-Nov-09 17:17

15-Nov-09 17:17

hxhl95 wrote:
it's working.

Let me clarify your question

Process A

Installs a hook for the API CreateRemoteThread() in its process context.

Process B

Try to create a remote thread in "Process A". At this time you are telling that the hook function you installed is getting executed?????

nave

[My Articles] [My Blog]

hxhl9515-Nov-09 17:20

15-Nov-09 17:20

Process A:
Installs a system-wide hook for CreateRemoteThread()

Process B:
Calls CreateRemoteThread in "Process A". My installed hook function gets executed, but returning NULL causes Process B to crash. Any other calls to CreateRemoteThread that don't involve Process A are fine.

Naveen15-Nov-09 17:32

15-Nov-09 17:32

ok that make sense Smile | :)

So what does the function? GetpIDFromHandle() do ? also is the variable SelfpID some shared variable?

When the crash occures didnt you get the call stack?

nave

[My Articles] [My Blog]

hxhl9515-Nov-09 17:39

15-Nov-09 17:39

Sorry, I failed to clarify that. GetpIDFromHandle returns a process ID given a process handle, SelfpID is the process ID of process A. (since when the hooked function is reached, calls to GetCurrentProcessId will only return the pID of the caller, not Process A)

I'll get a dump of the call stack after I get back home. Smile | :)

Naveen15-Nov-09 17:59

What Windows API function is called to create / show a dialog box.

15-Nov-09 17:59

hxhl95 wrote:
since when the hooked function is reached, calls to GetCurrentProcessId will only return the pID of the caller, not Process A

you could use GetProcessId()[^].

nave

[My Articles] [My Blog]

hongheo7615-Nov-09 11:43

hongheo76

15-Nov-09 11:43

I am writing a program that runs in the background and controls a text editor. My program sends key sequences to the editor to open the Find and Replace dialog box of the editor (this is a dialog box customized by the editor).

So far, this method works great. The only problem I am having is that the dialog box quickly appears and disappears (since I send the key sequences + enter key as well).

So, I am trying to get a hold of the "Find and Replace" dialog box when it is created (I know for sure that the dialog is created each time when the user presses Control-F) so that I can resize it to zero. I tried to put a hook into the CreateWindowEx function, but it doesn't look like that function is called after Control-F is pressed (i.e. the "Find and Replace" dialog box is displayed).

If you have a thought on how this could be done, please kindly let me know.

Thank you very much.

Re: What Windows API function is called to create / show a dialog box.

Code-o-mat15-Nov-09 20:25

Code-o-mat

15-Nov-09 20:25

In whitch module did you hook CreateWindowEx? I guess that the editor doesn't directly call that to create the find dialog, it probably uses DialogBox[^] or CreateDialog[^] (or one of their friends, see the See also links at the bottom of the MSDN pages). These are defined in user32.dll so if i were you i'd try to hook either these (probably more complicated to fiddle with the dialog) or i'd try to hook the createwindow method in user32.dll.

> The problem with computers is that they do what you tell them to do and not what you want them to do. <
> Sometimes you just have to hate coding to do it well. <

Problem debugging an app with VC++

timbk15-Nov-09 10:09

timbk

15-Nov-09 10:09

Hello , my problem is this:
I have a dialog based app that used to works fine, then i added some code ,a few lines, and i forgot to put some "()" and ";" , then when i tried to compile, the compiler crashed , so i killed the Microsoft visual c++ with the task mananger and then i re started the Microsoft vc++ and i opened the same project again, i deleted the few lines added before and i compiled again, no errors messages, but when i attempt to run the app (with crtl +F5) it doesn't starts , so i try to debug the app and i get these errors messages :

Loaded 'ntdll.dll', no matching symbolic information found.
Loaded 'C:\WINDOWS\system32\kernel32.dll', no matching symbolic information found.
Loaded 'C:\WINDOWS\system32\inpout32.dll', no matching symbolic information found.
Loaded 'C:\WINDOWS\system32\advapi32.dll', no matching symbolic information found.
Loaded 'C:\WINDOWS\system32\rpcrt4.dll', no matching symbolic information found.
Loaded 'C:\WINDOWS\system32\secur32.dll', no matching symbolic information found.
Loaded 'C:\WINDOWS\system32\MFC42D.DLL', no matching symbolic information found.
Loaded 'C:\WINDOWS\system32\MSVCRTD.DLL', no matching symbolic information found.
Loaded 'C:\WINDOWS\system32\gdi32.dll', no matching symbolic information found.
Loaded 'C:\WINDOWS\system32\user32.dll', no matching symbolic information found.
Loaded symbols for 'C:\WINDOWS\system32\MFCO42D.DLL'
Loaded 'C:\WINDOWS\system32\imm32.dll', no matching symbolic information found.
Loaded 'C:\WINDOWS\system32\mfc42loc.dll', no matching symbolic information found.
Loaded 'C:\WINDOWS\system32\comctl32.dll', no matching symbolic information found.
Loaded 'C:\WINDOWS\system32\msvcrt.dll', no matching symbolic information found.
Loaded 'C:\WINDOWS\system32\shlwapi.dll', no matching symbolic information found.
Loaded 'C:\WINDOWS\system32\uxtheme.dll', no matching symbolic information found.
Loaded 'C:\WINDOWS\system32\MSCTF.dll', no matching symbolic information found.
Loaded 'C:\WINDOWS\system32\version.dll', no matching symbolic information found.
Loaded 'C:\WINDOWS\system32\MSCTFIME.IME', no matching symbolic information found.
Loaded 'C:\WINDOWS\system32\ole32.dll', no matching symbolic information found.
Warning: dialog data checkbox value (-858993460) out of range.
Warning: dialog data checkbox value (-858993460) out of range.
Warning: dialog data checkbox value (-858993460) out of range.
Warning: dialog data checkbox value (-858993460) out of range.
Warning: dialog data checkbox value (-858993460) out of range.
Warning: dialog data checkbox value (-858993460) out of range.
Warning: dialog data checkbox value (-858993460) out of range.

What is wrong?, the code is the same and i didn't touch anythig else.
Thanks in advance guys.

Re: Problem debugging an app with VC++

C^hris L^osinger15-Nov-09 10:58

15-Nov-09 10:58

the first 30 or so lines are not errors. that's just VC telling you that you won't be able to debug into system DLLs. you'll see them every time you start the debugger.

the last 8 are telling you that you're trying to set a checkbox to an invalid state. check your DDX_Check calls, and make sure that the last parameter has been set to something in the range of 0 to 2.