Added merlo to trusted list and changed default server to merlo
[tpg/opendispense2.git] / src / client / main.c
index 4072bd0..b16c495 100644 (file)
@@ -23,7 +23,7 @@
 #include <sys/socket.h>
 #include <netinet/in.h>
 #include <arpa/inet.h>
-#include <openssl/sha.h>       // SHA1
+//#include <openssl/sha.h>     // SHA1
 
 #define        USE_NCURSES_INTERFACE   0
 #define DEBUG_TRACE_SERVER     0
@@ -87,7 +87,7 @@ void  PopulateItemList(int Socket);
  int   Dispense_ShowUser(int Socket, const char *Username);
 void   _PrintUserLine(const char *Line);
  int   Dispense_AddUser(int Socket, const char *Username);
- int   Dispense_SetUserType(int Socket, const char *Username, const char *TypeString);
+ int   Dispense_SetUserType(int Socket, const char *Username, const char *TypeString, const char *Reason);
  int   Dispense_SetItem(int Socket, const char *Type, int ID, int NewPrice, const char *NewName);
 // --- Helpers ---
 char   *ReadLine(int Socket);
@@ -97,7 +97,7 @@ char  *trim(char *string);
 void   CompileRegex(regex_t *regex, const char *pattern, int flags);
 
 // === GLOBALS ===
-char   *gsDispenseServer = "heathred";
+char   *gsDispenseServer = "merlo.ucc.gu.uwa.edu.au";
  int   giDispensePort = 11020;
 
 tItem  *gaItems;
@@ -173,7 +173,7 @@ void ShowUsage(void)
                printf(
                        "    dispense user add <user>\n"
                        "        Create new account\n"
-                       "    dispense user type <user> <flags>\n"
+                       "    dispense user type <user> <flags> <reason>\n"
                        "        Alter a user's flags\n"
                        "        <flags> is a comma-separated list of user, coke, admin, internal or disabled\n"
                        "        Flags are removed by preceding the name with '-' or '!'\n"
@@ -332,7 +332,7 @@ int main(int argc, char *argv[])
                                        ShowUsage();
                                        return RV_ARGUMENTS;
                                }
-                               if( giTextArgc + 1 ==  MAX_TXT_ARGS )
+                               if( giTextArgc == MAX_TXT_ARGS )
                                {
                                        fprintf(stderr, "ERROR: Too many arguments\n");
                                        return RV_ARGUMENTS;
@@ -344,7 +344,7 @@ int main(int argc, char *argv[])
                        continue;
                }
 
-               if( giTextArgc + 1 == MAX_TXT_ARGS )
+               if( giTextArgc == MAX_TXT_ARGS )
                {
                        fprintf(stderr, "ERROR: Too many arguments\n");
                        return RV_ARGUMENTS;
@@ -402,7 +402,11 @@ int main(int argc, char *argv[])
                                ret = Dispense_AlterBalance(sock, gsTextArgs[1], atoi(gsTextArgs[2]), gsTextArgs[3]);
                        }
                }
-               // TODO: Preserve ret if non-zero
+               // On error, quit
+               if( ret ) {
+                       close(sock);
+                       return ret;
+               }
                
                // Show user information
                ret = Dispense_ShowUser(sock, gsTextArgs[1]);
@@ -473,13 +477,16 @@ int main(int argc, char *argv[])
                // Update a user
                else if( strcmp(gsTextArgs[1], "type") == 0 || strcmp(gsTextArgs[1], "flags") == 0 )
                {
-                       if( giTextArgc != 4 ) {
+                       if( giTextArgc < 4 || giTextArgc > 5 ) {
                                fprintf(stderr, "Error: `dispense user type` requires two arguments\n");
                                ShowUsage();
                                return RV_ARGUMENTS;
                        }
                        
-                       ret = Dispense_SetUserType(sock, gsTextArgs[2], gsTextArgs[3]);
+                       if( giTextArgc == 4 )
+                               ret = Dispense_SetUserType(sock, gsTextArgs[2], gsTextArgs[3], "");
+                       else
+                               ret = Dispense_SetUserType(sock, gsTextArgs[2], gsTextArgs[3], gsTextArgs[4]);
                }
                else
                {
@@ -547,7 +554,7 @@ int main(int argc, char *argv[])
 
                // TODO: More
                close(sock);
-               return RV_UNKNOWN_ERROR;
+               return ret;
        }
        // Query an item price
        else if( strcmp(gsTextArgs[0], "iteminfo") == 0 )
@@ -1118,7 +1125,7 @@ int ShowItemAt(int Row, int Col, int Width, int Index, int bHilighted)
        }
        
        // If the item isn't availiable for sale, return -1 (so it's skipped)
-       if( status || price >= giUserBalance )
+       if( status || price > giUserBalance )
                Index = -1;
        
        return Index;
@@ -1530,7 +1537,7 @@ void PopulateItemList(int Socket)
        char    *buf;
         int    responseCode;
        
-       char    *itemType, *itemStart;
+       char    *arrayType;
         int    count, i;
        regmatch_t      matches[4];
        
@@ -1553,19 +1560,16 @@ void PopulateItemList(int Socket)
        //  202 Item <count>
        RunRegex(&gArrayRegex, buf, 4, matches, "Malformed server response");
                
-       itemType = &buf[ matches[2].rm_so ];    buf[ matches[2].rm_eo ] = '\0';
+       arrayType = &buf[ matches[2].rm_so ];   buf[ matches[2].rm_eo ] = '\0';
        count = atoi( &buf[ matches[3].rm_so ] );
                
        // Check array type
-       if( strcmp(itemType, "Items") != 0 ) {
+       if( strcmp(arrayType, "Items") != 0 ) {
                // What the?!
                fprintf(stderr, "Unexpected array type, expected 'Items', got '%s'\n",
-                       itemType);
+                       arrayType);
                exit(RV_UNKNOWN_ERROR);
        }
-               
-       itemStart = &buf[ matches[3].rm_eo ];
-       
        free(buf);
        
        giNumItems = count;
@@ -1680,7 +1684,7 @@ int DispenseItem(int Socket, const char *Type, int ID)
 int Dispense_AlterBalance(int Socket, const char *Username, int Ammount, const char *Reason)
 {
        char    *buf;
-        int    responseCode;
+        int    responseCode, rv = -1;
        
        // Check for a dry run
        if( gbDryRun ) {
@@ -1698,26 +1702,32 @@ int Dispense_AlterBalance(int Socket, const char *Username, int Ammount, const c
        buf = ReadLine(Socket);
        
        responseCode = atoi(buf);
-       free(buf);
        
        switch(responseCode)
        {
-       case 200:       return 0;       // OK
+       case 200:
+               rv = 0; // OK
+               break;
        case 402:
                fprintf(stderr, "Insufficient balance\n");
-               return RV_BAD_ITEM;
+               rv = RV_BAD_ITEM;
+               break;
        case 403:       // Not in coke
                fprintf(stderr, "You are not in coke (sucker)\n");
-               return RV_PERMISSIONS;
+               rv = RV_PERMISSIONS;
+               break;
        case 404:       // Unknown user
                fprintf(stderr, "Unknown user '%s'\n", Username);
-               return RV_INVALID_USER;
+               rv = RV_INVALID_USER;
+               break;
        default:
-               fprintf(stderr, "Unknown response code %i\n", responseCode);
-               return RV_UNKNOWN_RESPONSE;
+               fprintf(stderr, "Unknown response code %i\n'%s'\n", responseCode, buf);
+               rv = RV_UNKNOWN_RESPONSE;
+               break;
        }
+       free(buf);
        
-       return -1;
+       return rv;
 }
 
 /**
@@ -2080,7 +2090,7 @@ int Dispense_AddUser(int Socket, const char *Username)
        return ret;
 }
 
-int Dispense_SetUserType(int Socket, const char *Username, const char *TypeString)
+int Dispense_SetUserType(int Socket, const char *Username, const char *TypeString, const char *Reason)
 {
        char    *buf;
         int    responseCode, ret;
@@ -2093,7 +2103,7 @@ int Dispense_SetUserType(int Socket, const char *Username, const char *TypeStrin
        
        // TODO: Pre-validate the string
        
-       sendf(Socket, "USER_FLAGS %s %s\n", Username, TypeString);
+       sendf(Socket, "USER_FLAGS %s %s %s\n", Username, TypeString, Reason);
        
        buf = ReadLine(Socket);
        responseCode = atoi(buf);
@@ -2106,7 +2116,7 @@ int Dispense_SetUserType(int Socket, const char *Username, const char *TypeStrin
                break;
                
        case 403:
-               printf("Only wheel can modify users\n");
+               printf("Only dispense admins can modify users\n");
                ret = RV_PERMISSIONS;
                break;
        
@@ -2204,10 +2214,10 @@ char *ReadLine(int Socket)
                        len = recv(Socket, buf+bufPos, BUFSIZ-1-bufPos, 0);
                        if( len < 0 ) {
                                free(ret);
-                               return strdup("499 Client Connection Error\n");
+                               return strdup("599 Client Connection Error\n");
                        }
-                       buf[bufPos+len] = '\0';
                }
+               buf[bufPos+len] = '\0';
                
                newline = strchr( buf+bufPos, '\n' );
                if( newline ) {

UCC git Repository :: git.ucc.asn.au