C / C++ / MFC

Re: How to Delete a used file in C++?

20-Mar-09 2:59

yogeshs wrote:
but how i can delete if it using somewhere in project...

Define using?

"Old age is like a bank account. You withdraw later in life what you have deposited along the way." - Unknown

"Fireproof doesn't mean the fire will never come. It means when the fire comes that you will be able to withstand it." - Michael Simmons

Maximilien20-Mar-09 3:09

Maximilien

20-Mar-09 3:09

I assume "using" means that it's "open"ed and you still want to delete it nonetheless.

You could try "ShFileOperation".

This signature was proudly tested on animals.

Obtaining process name from process handle [modified]

hxhl9519-Mar-09 19:42

19-Mar-09 19:42

Hello everyone,

It's a really straightforward question this time: I have a process handle and I wish to detect if that handle is one to my app. I have a function where I receive a handle to a process, and if the handle is a handle to any other app but mine, I send a message to it.

I've tried using GetProcessImageFileName, but unfortunately it returns garbage data...

char processName[255];
GetProcessImageFileName(ProcessHandle,processName,256);
MessageBox(NULL,processName,"hehehaha",MB_ICONINFORMATION);

Which leads to another non-related question. You see, I'm trying to hook NtTerminateProcess and deny all requests to terminate my own process. The hook works fine, but when I call the real NtTerminateProcess from inside the hookproc the process it's called on crashes. I am completely at a loss of why this is happening and would appreciate it if someone explained it to me.

When it crashes the error is The instruction at "0x10002383" referenced memory at "0x0000000c". The memory could not be "read". It's always the same two addresses for any process.

Thanks a lot. Smile | :)

P.S. Please, please, PLEASE do not reply with a message asking "Why would you want to do such a thing". This is for educational purposes only, and I mean that. Laugh | :laugh:

modified on Friday, March 20, 2009 1:51 AM

«_Superman_»19-Mar-09 19:53

19-Mar-09 19:53

Check the return value of the function to see if returns an error.
The call GetLastError to get the error value.
Its possible that the process handle does not have the PROCESS_QUERY_INFORMATION access right.

«_Superman_»
I love work. It gives me something to do between weekends.

hxhl9519-Mar-09 19:59

19-Mar-09 19:59

Return value is 0 for the GetProcessImageFileName function. Even though it returns garbage data.

«_Superman_»19-Mar-09 20:03

Re: Obtaining process name from process handle [modified]

19-Mar-09 20:03

Are you using a UNICODE build?
If so declare the array as TCHAR instead of char.

Or you can try the GetModuleBaseName function.

«_Superman_»
I love work. It gives me something to do between weekends.

hxhl9520-Mar-09 5:56

20-Mar-09 5:56

Not UNICODE.

I'm going to give GetModuleBaseName a try, but according to msdn:

MSDN:

To retrieve the base name of the main executable module for a remote process, use the GetProcessImageFileName or QueryFullProcessImageName function to retrieve the module name and then use the strrchr function as described in the previous paragraph. This is more efficient and more reliable than calling GetModuleBaseName with a NULL module handle.

EDIT/update: GetModuleBaseName also returns garbage...

GetModuleBaseName(ProcessHandle,NULL,processName,1024);
MessageBox(NULL,processName,"test",MB_ICONINFORMATION);

modified on Friday, March 20, 2009 12:02 PM

«_Superman_»21-Mar-09 3:31

Re: Obtaining process name from process handle [modified]

21-Mar-09 3:31

You say that the error code is success and still you're getting garbage.
This is highly unlikely.
I believe you're not doing any error checking.
So its probably gone wrong somewhere else.

GetModuleBaseName for instance returns the length of the string copied.
What is this value?

«_Superman_»
I love work. It gives me something to do between weekends.

hxhl9521-Mar-09 8:12

21-Mar-09 8:12

Oh, I'm sorry. I thought GetModuleBaseName returns an error code instead of the length of the string copied. I assumed the 0 it returned meant success D'Oh! | :doh:

That explains the garbage data.

EDIT: I checked GetLastError, it's returning an invalid handle error. WTF | :WTF:

Might this be because I'm getting the handle from NtTerminateProcess?

modified on Saturday, March 21, 2009 2:37 PM

David Crow20-Mar-09 3:02

20-Mar-09 3:02

hxhl95 wrote:
I have a process handle and I wish to detect if that handle is one to my app.

Call GetProcessId(), and then call EnumWindows() to find the matching process id.

hxhl9520-Mar-09 5:54

20-Mar-09 5:54

error C2065: 'GetProcessId' : undeclared identifier, even though I've included windows.h. Does the fact that I'm using VC++ 6.0 make a difference?

David Crow20-Mar-09 6:00

20-Mar-09 6:00

hxhl95 wrote:
Does the fact that I'm using VC++ 6.0 make a difference?

Only if you need to define _WIN32_WINNT. Look in winbase.h for the appropriate value (if any).

See here for more.

hxhl9520-Mar-09 6:08

20-Mar-09 6:08

winbase.h doesn't contain a definition of GetProcessId. I tried defining it manually with DWORD WINAPI GetProcessId(__in HANDLE ProcessHandle), but then I get an unresolved external symbol error, even though I'm obviously including kernel32.lib...

David Crow20-Mar-09 7:16

20-Mar-09 7:16

hxhl95 wrote:
winbase.h doesn't contain a definition of GetProcessId.

You need to install the Platform SDK.

hxhl9520-Mar-09 7:23

20-Mar-09 7:23

I have.

David Crow20-Mar-09 7:31

20-Mar-09 7:31

So are you looking in C:\Program Files\Microsoft SDK\include\winbase.h instead of C:\Program Files\Microsoft Visual Studio\VC98\Include\winbase.h? If you add #include <winbase.h></winbase.h> to the top of a file, right-click it and select Open, which of the two files opens?

hxhl9520-Mar-09 8:04

20-Mar-09 8:04

I did

#include <C:\Program Files\Microsoft Platform SDK\Include\winbase.h><br />

just in case, and it's still giving me an undeclared error. When I open the file the SDK file opens, and I can see the definition of GetProcessId in the file. However the compiler just doesn't find it. Sigh | :sigh:

David Crow20-Mar-09 8:16

20-Mar-09 8:16

I'm out of suggestions at this point. This compiles fine for me:

#include <windows.h>
 
void main( void )
{
    GetProcessId(0);
}

hxhl9520-Mar-09 10:08

20-Mar-09 10:08

Okay. I'll just try a different approach if I can't get this working by tomorrow.

Thanks a lot for your help Smile | :)

MFC + Threading = C2665

Killiconn19-Mar-09 14:43

Killiconn

19-Mar-09 14:43

I have been surfing the web trying to find a solution and I've tried some of them and still can't compile. It is a standard VS generated DlgClass and I just want to launch the worker which is prototyped up top as global. I've tried casting ExecuteThread every way possible pretty much in the thread launch function. Any obvious solutions?

// Prototypes:
void ExecuteThread(LPVOID lParam)


// Button to launch the worker:
void CVisualEmulatorDlg::OnBnClickedRun()
{
	HWND hWnd = GetSafeHwnd();

	CWinThread *pThread = AfxBeginThread(ExecuteThread, hWnd, THREAD_PRIORITY_NORMAL);
	
	//cpu->Execute();
}

// Worker Thread:
void ExecuteThread(LPVOID lParam)
{

        //cpu->Execute();

	MessageBox( (HWND)lParam, (LPCWSTR)"Thread Start", (LPCWSTR)"Secondary Thread", MB_OK );
	return;
}

«_Superman_»19-Mar-09 19:03

19-Mar-09 19:03

You haven't been reading your documentation!!! Poke tongue | ;-P

Try this.

UINT __cdecl ExecuteThread( LPVOID pParam );

«_Superman_»
I love work. It gives me something to do between weekends.

Killiconn20-Mar-09 10:50

Killiconn

20-Mar-09 10:50

The reason I ended up trying void, along with any other things is that UNIT dosn't show up as a data type. Is there a header that needs to be included? Having trouble researching it because everything that comes back is about unit testing.

From one page it looked like UNIT meant use, int, long, or short, but tried and non of those worked. Tried with and without the __cdecl as well.

Never come across the UNIT before and wishing right now I never had. Code looks basically the same. Any ideas?

// Launcher
void CVisualEmulatorDlg::OnBnClickedRun()
{
	HWND hWnd = GetSafeHwnd();

	CWinThread *pThread = AfxBeginThread(ExecuteThread, hWnd, THREAD_PRIORITY_NORMAL);
	
	//cpu->Execute();
}

// Worker
UNIT __cdecl ExecuteThread(LPVOID lParam)
{
	MessageBox( (HWND)lParam, (LPCWSTR)"Thread Start", (LPCWSTR)"Secondary Thread", MB_OK );
}

«_Superman_»21-Mar-09 3:34

21-Mar-09 3:34

typo typo typo typo

Its UINT not UNIT.

«_Superman_»
I love work. It gives me something to do between weekends.