Yo lo dejaria mas o meno asi

void pass(char *nombre, char *mi_clave)
{
    const char clave[] = "contrasena";
    const char usuario[] = "usuario";
 
    int cierto = 0, intentos = 3;
 
    do
    {
        printf("\nIntroduzca nombre de usuario: ");
        scanf("%s", nombre);
 
        printf("\nIntroduzca la clave: ");
        scanf("%s", mi_clave);
 
        if(strcmp(clave, mi_clave)==0 && strcmp(usuario, nombre)==0) {
          printf("\nUsuario y/o contrasena validos");
          cierto = 1;
         }
        else if( intentos ) {
          printf("\nAcceso Denegado ...intentos restantes %d ", intentos);
          intentos--;
        }
        else {
          printf("\nNo tiene mas oportunidades para probar");
          break;
        }
     }
     while(!cierto);
 
}
 

Saludos  

Si apenas empiezas en tu aprendizaje del lenguaje C lo recomendado es evitar el uso de "fflush(stdin)", gets y la biblioteca conio de Borland. Mas información en la pagina |Lo que no hay que hacer en C/C++. Nivel basico|.

Dos errores en esa función son la falta de los nombres de los parámetros y no inicializar a cero la variable "h" antes del bucle donde se pide la contraseña. Esto ultimo es lo que provoca que la función no reconozca la contraseña como valida.

Tampoco es necesario inicializar una variable con un valor cuando lo primero que haces con ella es almacenar otro valor:

i=0;
 
/* ... */
 
for (i = 1; i <= 3; i++)

En un caso así hay que eliminar (por superflua) la primera asignación.

La función con las correcciones:

#include <stdio.h>
#include <stdlib.h>
#include <conio.h>
 
void val_pas(char const *nombre_ok, char const *contrasena_ok) ;
 
int main(void)
{
   val_pas("foo", "bar");
 
   return EXIT_SUCCESS;
}
 
 
void val_pas(char const *nombre_ok, char const *contrasena_ok)
{
   char nombre[100];
   char contrasena[100];
   int i;
   int h;
 
   for (i = 0; i < 3; i++){
      printf("Ingrese nombre de usuario: ");
      fflush(stdout);
      gets(nombre); /* Hay que cambiar por fgets */
 
      printf("Ingrese la contrasena: ");
      fflush(stdout);
      for (h = 0; (contrasena[h] = getch()) != '\r'; h++){
         putchar('*');
         fflush(stdout);
      }
      contrasena[h] = '\0';
      putchar('\n');
 
      if (strcmp(nombre, nombre_ok) == 0 && strcmp(contrasena, contrasena_ok) == 0)
         break;
      else
         puts("Nombre y/o contrasena invalidos");
   }
 
   if (i ==3){
      puts("No tiene mas oportunidades para probar");
      exit(EXIT_FAILURE);
   }
}

Faltan algunos cambios como la sustitución de gets por fgets, no lo agregue para no hacer mas largo el programa pero es algo que debes evitar (de nuevo hay que leer la pagina sobre recomendaciones).

Un saludo

Hola.

Depende si estas programando en C o C++.

Como C++ está basado en C, el compilador no se quejará si utilizas printf o scanf, etc. Aunque, si estas compilando en C++, es mejor utilizar las funciones proias de él.
La situación no es reversible, no se pueden utilizar funciones propias de C++ tales como cout o cin en C.

Saludos

En C cuando se requiere imprimir texto con formato se utiliza printf, por ejemplo:

int a = 123;
 
/* ... */
 
printf("El valor de a es %d\n", a);

Para imprimir una cadena literal terminada con un avance de linea se puede utilizar printf o puts, aquí es mejor la segunda ya que el carácter '\n' se imprime automáticamente después del texto:

printf("hola, mundo\n");
 
/* O bien */
puts("hola, mundo");

Un saludo