Skip to content

Commit b996516

Browse files
committed
testsuite: Throw error when AFP login fails
1 parent f51dd5f commit b996516

File tree

7 files changed

+66
-21
lines changed

7 files changed

+66
-21
lines changed

test/testsuite/T2_spectest.c

+13-4
Original file line numberDiff line numberDiff line change
@@ -297,6 +297,7 @@ void usage( char * av0 )
297297
int main( int ac, char **av )
298298
{
299299
int cc;
300+
int ret;
300301
static char *vers = "AFPVersion 2.1";
301302
static char *uam = "Cleartxt Passwrd";
302303

@@ -419,10 +420,14 @@ static char *uam = "Cleartxt Passwrd";
419420

420421
/* login */
421422
if (Version >= 30) {
422-
FPopenLoginExt(Conn, vers, uam, User, Password);
423+
ret = FPopenLoginExt(Conn, vers, uam, User, Password);
423424
}
424425
else {
425-
FPopenLogin(Conn, vers, uam, User, Password);
426+
ret = FPopenLogin(Conn, vers, uam, User, Password);
427+
}
428+
if (ret) {
429+
printf("Login failed\n");
430+
exit(1);
426431
}
427432
Conn->afp_version = Version;
428433

@@ -452,11 +457,15 @@ static char *uam = "Cleartxt Passwrd";
452457
}
453458
/* login */
454459
if (Version >= 30) {
455-
FPopenLoginExt(Conn2, vers, uam, User2, Password);
460+
ret = FPopenLoginExt(Conn2, vers, uam, User2, Password);
456461
}
457462
else {
458-
FPopenLogin(Conn2, vers, uam, User2, Password);
463+
ret = FPopenLogin(Conn2, vers, uam, User2, Password);
459464
}
465+
if (ret) {
466+
printf("Login failed\n");
467+
exit(1);
468+
}
460469
Conn2->afp_version = Version;
461470
}
462471
/*********************************

test/testsuite/afparg.c

+7-2
Original file line numberDiff line numberDiff line change
@@ -244,6 +244,7 @@ char *uam = "Cleartxt Passwrd";
244244
int main( int ac, char **av )
245245
{
246246
int cc;
247+
int ret;
247248

248249
while (( cc = getopt( ac, av, "v1234567h:p:s:u:w:f:l" )) != EOF ) {
249250
switch ( cc ) {
@@ -332,10 +333,14 @@ int cc;
332333

333334
/* login */
334335
if (Version >= 30) {
335-
FPopenLoginExt(Conn, vers, uam, User, Password);
336+
ret = FPopenLoginExt(Conn, vers, uam, User, Password);
336337
}
337338
else {
338-
FPopenLogin(Conn, vers, uam, User, Password);
339+
ret = FPopenLogin(Conn, vers, uam, User, Password);
340+
}
341+
if (ret) {
342+
printf("Login failed\n");
343+
exit(1);
339344
}
340345
Conn->afp_version = Version;
341346

test/testsuite/afpcli.c

-1
Original file line numberDiff line numberDiff line change
@@ -515,7 +515,6 @@ uint16_t len16;
515515
uint16_t temp16;
516516
int ofs;
517517
DSI *dsi;
518-
519518
dsi = &conn->dsi;
520519

521520
if (DSIOpenSession(conn)) {

test/testsuite/fail_spectest.c

+13-4
Original file line numberDiff line numberDiff line change
@@ -290,6 +290,7 @@ char *uam = "Cleartxt Passwrd";
290290
int main( int ac, char **av )
291291
{
292292
int cc;
293+
int ret;
293294

294295
while (( cc = getopt( ac, av, "v1234567h:H:p:s:u:d:w:c:f:lmx" )) != EOF ) {
295296
switch ( cc ) {
@@ -401,10 +402,14 @@ int cc;
401402

402403
/* login */
403404
if (Version >= 30) {
404-
FPopenLoginExt(Conn, vers, uam, User, Password);
405+
ret = FPopenLoginExt(Conn, vers, uam, User, Password);
405406
}
406407
else {
407-
FPopenLogin(Conn, vers, uam, User, Password);
408+
ret = FPopenLogin(Conn, vers, uam, User, Password);
409+
}
410+
if (ret) {
411+
printf("Login failed\n");
412+
exit(1);
408413
}
409414
Conn->afp_version = Version;
410415

@@ -434,11 +439,15 @@ int cc;
434439
}
435440
/* login */
436441
if (Version >= 30) {
437-
FPopenLoginExt(Conn2, vers, uam, User2, Password);
442+
ret = FPopenLoginExt(Conn2, vers, uam, User2, Password);
438443
}
439444
else {
440-
FPopenLogin(Conn2, vers, uam, User2, Password);
445+
ret = FPopenLogin(Conn2, vers, uam, User2, Password);
441446
}
447+
if (ret) {
448+
printf("Login failed\n");
449+
exit(1);
450+
}
442451
Conn2->afp_version = Version;
443452
}
444453
/*********************************

test/testsuite/rotest.c

+7-2
Original file line numberDiff line numberDiff line change
@@ -367,6 +367,7 @@ void usage( char * av0 )
367367
int main( int ac, char **av )
368368
{
369369
int cc;
370+
int ret;
370371
static char *uam = "Cleartxt Passwrd";
371372

372373
while (( cc = getopt( ac, av, "v1234567h:p:u:w:ms:" )) != EOF ) {
@@ -440,10 +441,14 @@ static char *uam = "Cleartxt Passwrd";
440441
/* ------------------------ */
441442
connect_server();
442443
if (Version >= 30) {
443-
FPopenLoginExt(Conn, vers, uam, User, Password);
444+
ret = FPopenLoginExt(Conn, vers, uam, User, Password);
444445
}
445446
else {
446-
FPopenLogin(Conn, vers, uam, User, Password);
447+
ret = FPopenLogin(Conn, vers, uam, User, Password);
448+
}
449+
if (ret) {
450+
printf("Login failed\n");
451+
exit(1);
447452
}
448453
Conn->afp_version = Version;
449454
run_all();

test/testsuite/sleeptest.c

+13-4
Original file line numberDiff line numberDiff line change
@@ -195,6 +195,7 @@ char *uam = "Cleartxt Passwrd";
195195
int main( int ac, char **av )
196196
{
197197
int cc;
198+
int ret;
198199

199200
while (( cc = getopt( ac, av, "v1234567ah:H:p:s:S:u:d:w:c:f:Llmxi" )) != EOF ) {
200201
switch ( cc ) {
@@ -319,10 +320,14 @@ int cc;
319320

320321
/* login */
321322
if (Version >= 30) {
322-
FPopenLoginExt(Conn, vers, uam, User, Password);
323+
ret = FPopenLoginExt(Conn, vers, uam, User, Password);
323324
}
324325
else {
325-
FPopenLogin(Conn, vers, uam, User, Password);
326+
ret = FPopenLogin(Conn, vers, uam, User, Password);
327+
}
328+
if (ret) {
329+
printf("Login failed\n");
330+
exit(1);
326331
}
327332
Conn->afp_version = Version;
328333

@@ -352,11 +357,15 @@ int cc;
352357
}
353358
/* login */
354359
if (Version >= 30) {
355-
FPopenLoginExt(Conn2, vers, uam, User2, Password);
360+
ret = FPopenLoginExt(Conn2, vers, uam, User2, Password);
356361
}
357362
else {
358-
FPopenLogin(Conn2, vers, uam, User2, Password);
363+
ret = FPopenLogin(Conn2, vers, uam, User2, Password);
359364
}
365+
if (ret) {
366+
printf("Login failed\n");
367+
exit(1);
368+
}
360369
Conn2->afp_version = Version;
361370
}
362371
/*********************************

test/testsuite/spectest.c

+13-4
Original file line numberDiff line numberDiff line change
@@ -318,6 +318,7 @@ char *uam = "Cleartxt Passwrd";
318318
int main( int ac, char **av )
319319
{
320320
int cc;
321+
int ret;
321322

322323
while (( cc = getopt( ac, av, "v1234567ah:H:p:s:S:u:d:w:c:f:Llmxi" )) != EOF ) {
323324
switch ( cc ) {
@@ -446,10 +447,14 @@ int cc;
446447

447448
/* login */
448449
if (Version >= 30) {
449-
FPopenLoginExt(Conn, vers, uam, User, Password);
450+
ret = FPopenLoginExt(Conn, vers, uam, User, Password);
450451
}
451452
else {
452-
FPopenLogin(Conn, vers, uam, User, Password);
453+
ret = FPopenLogin(Conn, vers, uam, User, Password);
454+
}
455+
if (ret) {
456+
printf("Login failed\n");
457+
exit(1);
453458
}
454459
Conn->afp_version = Version;
455460

@@ -479,11 +484,15 @@ int cc;
479484
}
480485
/* login */
481486
if (Version >= 30) {
482-
FPopenLoginExt(Conn2, vers, uam, User2, Password);
487+
ret = FPopenLoginExt(Conn2, vers, uam, User2, Password);
483488
}
484489
else {
485-
FPopenLogin(Conn2, vers, uam, User2, Password);
490+
ret = FPopenLogin(Conn2, vers, uam, User2, Password);
486491
}
492+
if (ret) {
493+
printf("Login failed\n");
494+
exit(1);
495+
}
487496
Conn2->afp_version = Version;
488497
}
489498
/*********************************

0 commit comments

Comments
 (0)