scanf only reads first field












0















I want to read in two strings with scanf and store them in two variables. How ever only the first one seems to be read properly, the second one just returns (null), but I'm not sure why though.



int main(int argc, char **argv) {

  char *c, *p;



  printf("Enter a command with a parameter: ");

  scanf("%s%s", c, p);

  ...

}


Somebody got an idea, what's going wrong? I don't find any errors and for my understanding it should work.






c scanf stdio







share|improve this question























  • When I enter ls -l it only reads the ls properly, that's what i don't understand. scanf should read the white space between them and store the -l in p, but i doesn't.

    – anonymoose
    Nov 19 '18 at 15:39








  • 3





    You've not initialized either c or p to point to anywhere, so you have undefined behaviour.

    – Jonathan Leffler
    Nov 19 '18 at 15:42











  • It’s undefined behavior you didn’t initialize the char * variables.

    – danglingpointer
    Nov 19 '18 at 15:42











  • You don't look at the return value of scanf. How do you know how many fields were parsed?

    – Gerhardh
    Nov 19 '18 at 15:50











  • FYI: There's a Q&A How to prevent scanf() causing buffer overflow in C?

    – Jonathan Leffler
    Nov 19 '18 at 15:54
















0















I want to read in two strings with scanf and store them in two variables. How ever only the first one seems to be read properly, the second one just returns (null), but I'm not sure why though.



int main(int argc, char **argv) {

  char *c, *p;



  printf("Enter a command with a parameter: ");

  scanf("%s%s", c, p);

  ...

}


Somebody got an idea, what's going wrong? I don't find any errors and for my understanding it should work.






c scanf stdio







share|improve this question























  • When I enter ls -l it only reads the ls properly, that's what i don't understand. scanf should read the white space between them and store the -l in p, but i doesn't.

    – anonymoose
    Nov 19 '18 at 15:39








  • 3





    You've not initialized either c or p to point to anywhere, so you have undefined behaviour.

    – Jonathan Leffler
    Nov 19 '18 at 15:42











  • It’s undefined behavior you didn’t initialize the char * variables.

    – danglingpointer
    Nov 19 '18 at 15:42











  • You don't look at the return value of scanf. How do you know how many fields were parsed?

    – Gerhardh
    Nov 19 '18 at 15:50











  • FYI: There's a Q&A How to prevent scanf() causing buffer overflow in C?

    – Jonathan Leffler
    Nov 19 '18 at 15:54














0












0








0








I want to read in two strings with scanf and store them in two variables. How ever only the first one seems to be read properly, the second one just returns (null), but I'm not sure why though.



int main(int argc, char **argv) {

  char *c, *p;



  printf("Enter a command with a parameter: ");

  scanf("%s%s", c, p);

  ...

}


Somebody got an idea, what's going wrong? I don't find any errors and for my understanding it should work.






c scanf stdio







share|improve this question














I want to read in two strings with scanf and store them in two variables. How ever only the first one seems to be read properly, the second one just returns (null), but I'm not sure why though.



int main(int argc, char **argv) {

  char *c, *p;



  printf("Enter a command with a parameter: ");

  scanf("%s%s", c, p);

  ...

}


Somebody got an idea, what's going wrong? I don't find any errors and for my understanding it should work.






c scanf stdio




c scanf stdio






share|improve this question













share|improve this question











share|improve this question




share|improve this question










asked Nov 19 '18 at 15:34









anonymooseanonymoose

335




335













  • When I enter ls -l it only reads the ls properly, that's what i don't understand. scanf should read the white space between them and store the -l in p, but i doesn't.

    – anonymoose
    Nov 19 '18 at 15:39








  • 3





    You've not initialized either c or p to point to anywhere, so you have undefined behaviour.

    – Jonathan Leffler
    Nov 19 '18 at 15:42











  • It’s undefined behavior you didn’t initialize the char * variables.

    – danglingpointer
    Nov 19 '18 at 15:42











  • You don't look at the return value of scanf. How do you know how many fields were parsed?

    – Gerhardh
    Nov 19 '18 at 15:50











  • FYI: There's a Q&A How to prevent scanf() causing buffer overflow in C?

    – Jonathan Leffler
    Nov 19 '18 at 15:54



















  • When I enter ls -l it only reads the ls properly, that's what i don't understand. scanf should read the white space between them and store the -l in p, but i doesn't.

    – anonymoose
    Nov 19 '18 at 15:39








  • 3





    You've not initialized either c or p to point to anywhere, so you have undefined behaviour.

    – Jonathan Leffler
    Nov 19 '18 at 15:42











  • It’s undefined behavior you didn’t initialize the char * variables.

    – danglingpointer
    Nov 19 '18 at 15:42











  • You don't look at the return value of scanf. How do you know how many fields were parsed?

    – Gerhardh
    Nov 19 '18 at 15:50











  • FYI: There's a Q&A How to prevent scanf() causing buffer overflow in C?

    – Jonathan Leffler
    Nov 19 '18 at 15:54

















When I enter ls -l it only reads the ls properly, that's what i don't understand. scanf should read the white space between them and store the -l in p, but i doesn't.

– anonymoose
Nov 19 '18 at 15:39







When I enter ls -l it only reads the ls properly, that's what i don't understand. scanf should read the white space between them and store the -l in p, but i doesn't.

– anonymoose
Nov 19 '18 at 15:39






3




3





You've not initialized either c or p to point to anywhere, so you have undefined behaviour.

– Jonathan Leffler
Nov 19 '18 at 15:42





You've not initialized either c or p to point to anywhere, so you have undefined behaviour.

– Jonathan Leffler
Nov 19 '18 at 15:42













It’s undefined behavior you didn’t initialize the char * variables.

– danglingpointer
Nov 19 '18 at 15:42





It’s undefined behavior you didn’t initialize the char * variables.

– danglingpointer
Nov 19 '18 at 15:42













You don't look at the return value of scanf. How do you know how many fields were parsed?

– Gerhardh
Nov 19 '18 at 15:50





You don't look at the return value of scanf. How do you know how many fields were parsed?

– Gerhardh
Nov 19 '18 at 15:50













FYI: There's a Q&A How to prevent scanf() causing buffer overflow in C?

– Jonathan Leffler
Nov 19 '18 at 15:54





FYI: There's a Q&A How to prevent scanf() causing buffer overflow in C?

– Jonathan Leffler
Nov 19 '18 at 15:54












3 Answers
3






active

oldest

votes


















6














char *c, *p;


These are just pointers without any memory behind them. Moreover, they are uninitialized, so reading or assigning they value or reading from or assigning to the memory they point to would be undefined behavior and will spawn nasal demons.



In order to read input from user, you need a place to save the memory to.



char c[20], p[20];


Will allocate 20 characters for c and 20 character for p. c and p are not pointers, but arrays char[20]. However arrays handlers "decay" (read: magically change) into pointers in most contexts.



scanf for "%s" scanf modifier expects a pointer to char char* with enough memory to hold the inputted string.



I would do:



int main(int argc, char **argv) {

  char c[20], p[20];



  printf("Enter a command with a parameter: ");

  scanf("%19s%19s", c, p);

}


The 19 in the %19s limits scanf so that it does not reads input into memory region that is out of bounds in c and/or p pointers, which will lead to undefined behaviour too.






share|improve this answer





















  • 1





    Thanks a lot, i didn't even knew about the limitation stuff.

    – anonymoose
    Nov 19 '18 at 15:49













  • @JonathanLeffler Or just remove the comment, since it's no longer needed. :)

    – Broman
    Nov 19 '18 at 15:51



















4














This is undefined behavior. You have not initialized your pointers. Your compiler should tell you about this.



k.c:8:17: warning: variable 'c' is uninitialized when used here [-Wuninitialized]

  scanf("%s%s", c, p);


What you need to do is to initialize your pointers. Using malloc is one way of doing it.



char *c=malloc(SIZE);

char *p=malloc(SIZE);


You want to avoid undefined behavior at ALL COST because it means that the behavior can be ANYTHING, including working the way you want. It can make debugging a nightmare.



But as Swordfish mentioned in the comments, it is suicidal to do this without specifying a max length for your string, because you will have no control over if you write past your buffer, which also is undefined behavior.






share|improve this answer

































    0














    you can also use %ms instead of %s because it allocate memory itself and there is no need of allocation memory manually by programmer.



    int main(int argc, char **argv) {
    
  char *c, *p;
    

    
  printf("Enter a command with a parameter: ");
    
  scanf("%ms%ms", &c, &p); //no need of allocating memory manually
    
  ... 
    
}





    share|improve this answer



















    • 1





      "%ms" is a useful idea with libraries that employ this non-standard enhancement. difference between %ms and %s scanf. Note that code still should later free(c), free(p)

      – chux
      Nov 19 '18 at 18:47













    Your Answer






    StackExchange.ifUsing("editor", function () {
    StackExchange.using("externalEditor", function () {
    StackExchange.using("snippets", function () {
    StackExchange.snippets.init();
    });
    });
    }, "code-snippets");

    StackExchange.ready(function() {
    var channelOptions = {
    tags: "".split(" "),
    id: "1"
    };
    initTagRenderer("".split(" "), "".split(" "), channelOptions);

    StackExchange.using("externalEditor", function() {
    // Have to fire editor after snippets, if snippets enabled
    if (StackExchange.settings.snippets.snippetsEnabled) {
    StackExchange.using("snippets", function() {
    createEditor();
    });
    }
    else {
    createEditor();
    }
    });

    function createEditor() {
    StackExchange.prepareEditor({
    heartbeatType: 'answer',
    autoActivateHeartbeat: false,
    convertImagesToLinks: true,
    noModals: true,
    showLowRepImageUploadWarning: true,
    reputationToPostImages: 10,
    bindNavPrevention: true,
    postfix: "",
    imageUploader: {
    brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
    contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
    allowUrls: true
    },
    onDemand: true,
    discardSelector: ".discard-answer"
    ,immediatelyShowMarkdownHelp:true
    });


    }
    });














    draft saved

    draft discarded


















    StackExchange.ready(
    function () {
    StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53377940%2fscanf-only-reads-first-field%23new-answer', 'question_page');
    }
    );

    Post as a guest















    Required, but never shown

























    3 Answers
    3






    active

    oldest

    votes








    3 Answers
    3






    active

    oldest

    votes









    active

    oldest

    votes






    active

    oldest

    votes









    6














    char *c, *p;


    These are just pointers without any memory behind them. Moreover, they are uninitialized, so reading or assigning they value or reading from or assigning to the memory they point to would be undefined behavior and will spawn nasal demons.



    In order to read input from user, you need a place to save the memory to.



    char c[20], p[20];


    Will allocate 20 characters for c and 20 character for p. c and p are not pointers, but arrays char[20]. However arrays handlers "decay" (read: magically change) into pointers in most contexts.



    scanf for "%s" scanf modifier expects a pointer to char char* with enough memory to hold the inputted string.



    I would do:



    int main(int argc, char **argv) {
    
  char c[20], p[20];
    

    
  printf("Enter a command with a parameter: ");
    
  scanf("%19s%19s", c, p);
    
}


    The 19 in the %19s limits scanf so that it does not reads input into memory region that is out of bounds in c and/or p pointers, which will lead to undefined behaviour too.






    share|improve this answer





















    • 1





      Thanks a lot, i didn't even knew about the limitation stuff.

      – anonymoose
      Nov 19 '18 at 15:49













    • @JonathanLeffler Or just remove the comment, since it's no longer needed. :)

      – Broman
      Nov 19 '18 at 15:51
















    6














    char *c, *p;


    These are just pointers without any memory behind them. Moreover, they are uninitialized, so reading or assigning they value or reading from or assigning to the memory they point to would be undefined behavior and will spawn nasal demons.



    In order to read input from user, you need a place to save the memory to.



    char c[20], p[20];


    Will allocate 20 characters for c and 20 character for p. c and p are not pointers, but arrays char[20]. However arrays handlers "decay" (read: magically change) into pointers in most contexts.



    scanf for "%s" scanf modifier expects a pointer to char char* with enough memory to hold the inputted string.



    I would do:



    int main(int argc, char **argv) {
    
  char c[20], p[20];
    

    
  printf("Enter a command with a parameter: ");
    
  scanf("%19s%19s", c, p);
    
}


    The 19 in the %19s limits scanf so that it does not reads input into memory region that is out of bounds in c and/or p pointers, which will lead to undefined behaviour too.






    share|improve this answer





















    • 1





      Thanks a lot, i didn't even knew about the limitation stuff.

      – anonymoose
      Nov 19 '18 at 15:49













    • @JonathanLeffler Or just remove the comment, since it's no longer needed. :)

      – Broman
      Nov 19 '18 at 15:51














    6












    6








    6







    char *c, *p;


    These are just pointers without any memory behind them. Moreover, they are uninitialized, so reading or assigning they value or reading from or assigning to the memory they point to would be undefined behavior and will spawn nasal demons.



    In order to read input from user, you need a place to save the memory to.



    char c[20], p[20];


    Will allocate 20 characters for c and 20 character for p. c and p are not pointers, but arrays char[20]. However arrays handlers "decay" (read: magically change) into pointers in most contexts.



    scanf for "%s" scanf modifier expects a pointer to char char* with enough memory to hold the inputted string.



    I would do:



    int main(int argc, char **argv) {
    
  char c[20], p[20];
    

    
  printf("Enter a command with a parameter: ");
    
  scanf("%19s%19s", c, p);
    
}


    The 19 in the %19s limits scanf so that it does not reads input into memory region that is out of bounds in c and/or p pointers, which will lead to undefined behaviour too.






    share|improve this answer















    char *c, *p;


    These are just pointers without any memory behind them. Moreover, they are uninitialized, so reading or assigning they value or reading from or assigning to the memory they point to would be undefined behavior and will spawn nasal demons.



    In order to read input from user, you need a place to save the memory to.



    char c[20], p[20];


    Will allocate 20 characters for c and 20 character for p. c and p are not pointers, but arrays char[20]. However arrays handlers "decay" (read: magically change) into pointers in most contexts.



    scanf for "%s" scanf modifier expects a pointer to char char* with enough memory to hold the inputted string.



    I would do:



    int main(int argc, char **argv) {
    
  char c[20], p[20];
    

    
  printf("Enter a command with a parameter: ");
    
  scanf("%19s%19s", c, p);
    
}


    The 19 in the %19s limits scanf so that it does not reads input into memory region that is out of bounds in c and/or p pointers, which will lead to undefined behaviour too.







    share|improve this answer














    share|improve this answer



    share|improve this answer








    edited Nov 19 '18 at 15:49

























    answered Nov 19 '18 at 15:43









    Kamil CukKamil Cuk

    9,9611527




    9,9611527








    • 1





      Thanks a lot, i didn't even knew about the limitation stuff.

      – anonymoose
      Nov 19 '18 at 15:49













    • @JonathanLeffler Or just remove the comment, since it's no longer needed. :)

      – Broman
      Nov 19 '18 at 15:51














    • 1





      Thanks a lot, i didn't even knew about the limitation stuff.

      – anonymoose
      Nov 19 '18 at 15:49













    • @JonathanLeffler Or just remove the comment, since it's no longer needed. :)

      – Broman
      Nov 19 '18 at 15:51








    1




    1





    Thanks a lot, i didn't even knew about the limitation stuff.

    – anonymoose
    Nov 19 '18 at 15:49







    Thanks a lot, i didn't even knew about the limitation stuff.

    – anonymoose
    Nov 19 '18 at 15:49















    @JonathanLeffler Or just remove the comment, since it's no longer needed. :)

    – Broman
    Nov 19 '18 at 15:51





    @JonathanLeffler Or just remove the comment, since it's no longer needed. :)

    – Broman
    Nov 19 '18 at 15:51













    4














    This is undefined behavior. You have not initialized your pointers. Your compiler should tell you about this.



    k.c:8:17: warning: variable 'c' is uninitialized when used here [-Wuninitialized]
    
  scanf("%s%s", c, p);


    What you need to do is to initialize your pointers. Using malloc is one way of doing it.



    char *c=malloc(SIZE);
    
char *p=malloc(SIZE);


    You want to avoid undefined behavior at ALL COST because it means that the behavior can be ANYTHING, including working the way you want. It can make debugging a nightmare.



    But as Swordfish mentioned in the comments, it is suicidal to do this without specifying a max length for your string, because you will have no control over if you write past your buffer, which also is undefined behavior.






    share|improve this answer






























      4














      This is undefined behavior. You have not initialized your pointers. Your compiler should tell you about this.



      k.c:8:17: warning: variable 'c' is uninitialized when used here [-Wuninitialized]
      
  scanf("%s%s", c, p);


      What you need to do is to initialize your pointers. Using malloc is one way of doing it.



      char *c=malloc(SIZE);
      
char *p=malloc(SIZE);


      You want to avoid undefined behavior at ALL COST because it means that the behavior can be ANYTHING, including working the way you want. It can make debugging a nightmare.



      But as Swordfish mentioned in the comments, it is suicidal to do this without specifying a max length for your string, because you will have no control over if you write past your buffer, which also is undefined behavior.






      share|improve this answer




























        4












        4








        4







        This is undefined behavior. You have not initialized your pointers. Your compiler should tell you about this.



        k.c:8:17: warning: variable 'c' is uninitialized when used here [-Wuninitialized]
        
  scanf("%s%s", c, p);


        What you need to do is to initialize your pointers. Using malloc is one way of doing it.



        char *c=malloc(SIZE);
        
char *p=malloc(SIZE);


        You want to avoid undefined behavior at ALL COST because it means that the behavior can be ANYTHING, including working the way you want. It can make debugging a nightmare.



        But as Swordfish mentioned in the comments, it is suicidal to do this without specifying a max length for your string, because you will have no control over if you write past your buffer, which also is undefined behavior.






        share|improve this answer















        This is undefined behavior. You have not initialized your pointers. Your compiler should tell you about this.



        k.c:8:17: warning: variable 'c' is uninitialized when used here [-Wuninitialized]
        
  scanf("%s%s", c, p);


        What you need to do is to initialize your pointers. Using malloc is one way of doing it.



        char *c=malloc(SIZE);
        
char *p=malloc(SIZE);


        You want to avoid undefined behavior at ALL COST because it means that the behavior can be ANYTHING, including working the way you want. It can make debugging a nightmare.



        But as Swordfish mentioned in the comments, it is suicidal to do this without specifying a max length for your string, because you will have no control over if you write past your buffer, which also is undefined behavior.







        share|improve this answer














        share|improve this answer



        share|improve this answer








        edited Nov 19 '18 at 15:46

























        answered Nov 19 '18 at 15:41









        BromanBroman

        6,256112241




        6,256112241























            0














            you can also use %ms instead of %s because it allocate memory itself and there is no need of allocation memory manually by programmer.



            int main(int argc, char **argv) {
            
  char *c, *p;
            

            
  printf("Enter a command with a parameter: ");
            
  scanf("%ms%ms", &c, &p); //no need of allocating memory manually
            
  ... 
            
}





            share|improve this answer



















            • 1





              "%ms" is a useful idea with libraries that employ this non-standard enhancement. difference between %ms and %s scanf. Note that code still should later free(c), free(p)

              – chux
              Nov 19 '18 at 18:47


















            0














            you can also use %ms instead of %s because it allocate memory itself and there is no need of allocation memory manually by programmer.



            int main(int argc, char **argv) {
            
  char *c, *p;
            

            
  printf("Enter a command with a parameter: ");
            
  scanf("%ms%ms", &c, &p); //no need of allocating memory manually
            
  ... 
            
}





            share|improve this answer



















            • 1





              "%ms" is a useful idea with libraries that employ this non-standard enhancement. difference between %ms and %s scanf. Note that code still should later free(c), free(p)

              – chux
              Nov 19 '18 at 18:47
















            0












            0








            0







            you can also use %ms instead of %s because it allocate memory itself and there is no need of allocation memory manually by programmer.



            int main(int argc, char **argv) {
            
  char *c, *p;
            

            
  printf("Enter a command with a parameter: ");
            
  scanf("%ms%ms", &c, &p); //no need of allocating memory manually
            
  ... 
            
}





            share|improve this answer













            you can also use %ms instead of %s because it allocate memory itself and there is no need of allocation memory manually by programmer.



            int main(int argc, char **argv) {
            
  char *c, *p;
            

            
  printf("Enter a command with a parameter: ");
            
  scanf("%ms%ms", &c, &p); //no need of allocating memory manually
            
  ... 
            
}






            share|improve this answer












            share|improve this answer



            share|improve this answer










            answered Nov 19 '18 at 16:51









            Susheel DwivediSusheel Dwivedi

            554




            554








            • 1





              "%ms" is a useful idea with libraries that employ this non-standard enhancement. difference between %ms and %s scanf. Note that code still should later free(c), free(p)

              – chux
              Nov 19 '18 at 18:47
















            • 1





              "%ms" is a useful idea with libraries that employ this non-standard enhancement. difference between %ms and %s scanf. Note that code still should later free(c), free(p)

              – chux
              Nov 19 '18 at 18:47










            1




            1





            "%ms" is a useful idea with libraries that employ this non-standard enhancement. difference between %ms and %s scanf. Note that code still should later free(c), free(p)

            – chux
            Nov 19 '18 at 18:47







            "%ms" is a useful idea with libraries that employ this non-standard enhancement. difference between %ms and %s scanf. Note that code still should later free(c), free(p)

            – chux
            Nov 19 '18 at 18:47




















            draft saved

            draft discarded




















































            Thanks for contributing an answer to Stack Overflow!


            • Please be sure to answer the question. Provide details and share your research!

            But avoid …



            • Asking for help, clarification, or responding to other answers.

            • Making statements based on opinion; back them up with references or personal experience.


            To learn more, see our tips on writing great answers.




            draft saved


            draft discarded














            StackExchange.ready(
            function () {
            StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53377940%2fscanf-only-reads-first-field%23new-answer', 'question_page');
            }
            );

            Post as a guest















            Required, but never shown





















































            Required, but never shown














            Required, but never shown












            Required, but never shown







            Required, but never shown

































            Required, but never shown














            Required, but never shown












            Required, but never shown







            Required, but never shown







            -

            Popular posts from this blog

            Biblatex bibliography style without URLs when DOI exists (in Overleaf with Zotero bibliography)

            Image
            2 2 I have a Zotero bibliography in Overleaf. I use biblatex . usepackage[backend=biber,citestyle=authoryear]{biblatex} addbibresource{zoteroALH.bib} Bibliography entries look like: @book{de_saint-gervais_uniformization_2016, title = {Uniformization of {Riemann} {Surfaces}}, isbn = {978-3-03719-145-3}, url = {https://www.ems-ph.org/books/book.php?proj_nr=198&srch=series%7Chem}, urldate = {2019-02-03}, author = {de Saint-Gervais, Paul Henri}, year = {2016}, doi = {10.4171/145} } In the bibliography down the paper the reference shows like this: There is the DOI, great, but also the URL which is unnecessary when the DOI is there. How can I remove the URL when the DOI is present? biblatex bibliographies overleaf zote
            Read more

            ComboBox Display Member on multiple fields

            Image
            2 I'm trying to set the .DisplayMember property of a ComboBox in C#, but I want to bind it to multiple columns in the .DataSouce . My SQL looks like this: SELECT PersNbr, PersFirstName, PersMiddleName, PersLastName FROM Pers WHERE PersNbr = :persNbr; I'm saving this query in a DataTable so each column selected has it's own column in the Datatable . I want to make the .DisplayMember a combination of PersFirstName + PersMiddleName + PersLastName so their full name appears like this: comboBox.DisplayMemeber = "PersFirstName" + "PersMiddleName" + "PersLastName" I know I can just to this on the query: SELECT PersNbr, (PersFirstName || PersMiddleName || PersLastName) PersName and then just do this: comboBox.DisplayMember = "PersName"; but I
            Read more

            Is it possible to collect Nectar points via Trainline?

            Image
            2 Since I am not eligible for any Railcard, the only available discount/bonus scheme is Nectar Card for me. GWR, SWR, LNER and Virgin Trains allow to collect Nectar points and I have already linked a card to my GWR account. A friend suggested me to use Trainline App, because it can search for bus tickets either. I had a look at the application, but could not see any option for Nectar. Is there a way to collect Nectar points when I buy tickets via Trainline? uk trains loyalty-programs trainline share | improve this question asked Dec 2 '18 at 15:50 ahmedus 3,224 5 19 49
            Read more