automatically run program on startup

Question

0.00/5 (No votes)

See more:

I have written a small program which insert the value and its corresponding value into the Windows registry key.

Program is working fine but it is not inserting value and its corresponding value.

And one more thing when I run prog as an administrator RegSetValueEx() fails..but still inssert only the value not its data.

Please help for finding out the issue here.

My code is as follows..

C

#define WIN32_LEAN_AND_MEAN
#define WIN32_DEFAULT_LIBS
#ifndef _WIN32_WINNT
#define _WIN32_WINNT (0x0601)
#endif  /* _WIN32_WINNT */
#include <windows.h>
#include <stdio.h>
#include <conio.h>
#include <stdlib.h>
#include <tchar.h>
#include <unistd.h>
#include <stdbool.h>

#include<string.h>

    BOOL InstallRunOnStartup()
    {
      HKEY key;

      long result;
      BOOL ret = FALSE;
      LPTSTR val=L"12as3d12";
        LPTSTR a=L"zzz";

      TCHAR szBuf[20];


      result = RegOpenKeyEx(HKEY_CURRENT_USER, "SOFTWARE\\Microsoft\\Windows\\CurrentVersion\\Run", NULL, KEY_WRITE | KEY_WOW64_64KEY | KEY_SET_VALUE , &key);
      if (result == ERROR_SUCCESS)
      {

        printf("hi \n");
          if (RegSetValueEx(key, a, 0, REG_SZ,(LPBYTE)val, (DWORD)(lstrlen(val)+1) == ERROR_SUCCESS)){
            printf("success \n");
            ret = TRUE;
          }
            RegCloseKey(key);
      }
      return ret;
    }




    int main()
    {
        InstallRunOnStartup();
        getch();
    }

Posted 26-Sep-13 20:26pm

Raghav Gaur

Updated 26-Sep-13 22:13pm

Nelek

v2

Add a Solution

1 solution

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Jochen Arndt · Answer 1 · 2013-09-26T21:45:00

You are mixing Unicode and ANSI:

With ANSI builds, you are assigning Unicode strings to char* for your val and a variables.

With Unicode builds, you are passing an ANSI string to RegOpenKeyEx().

With ANSI builds, you are passing Unicode strings for value name and value to RegSetValueEx().

With Unicode builds, the RegSetValueEx() cbData parameter is wrong (the total length must be multiplied with sizeof(WHACR)).

Solution:
Use Unicode setting dependend macros and functions:

LPTSTR val=_T("12as3d12");
LPTSTR a=_T("zzz");
RegOpenKeyEx(HKEY_CURRENT_USER, _T("SOFTWARE\\Microsoft\\Windows\\CurrentVersion\\Run)", // ...
RegSetValueEx(key, a, 0, REG_SZ,(LPBYTE)val, (DWORD)(_tcslen(val)+1) * sizeof(TCHAR));