diff for duplicates of <87hcnq9mf2.fsf@wanadoo.fr> diff --git a/a/1.txt b/N1/1.txt index 7e2aee5..71d6d76 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -11,12 +11,12 @@ Al Viro <viro@ftp.linux.org.uk> writes: >> >> /* allocate some common virtual space to map the new pages */ >> area = get_vm_area(size, VM_ALLOC); ->> - if (area = 0) { ->> + if (area = NULL) { +>> - if (area == 0) { +>> + if (area == NULL) { >> free_pages(page, order); >> return NULL; > > Same comment about comparisons with NULL after allocation... I don't understand. pte is a pointer right ? So why should we -keep the = 0 ? +keep the == 0 ? diff --git a/a/content_digest b/N1/content_digest index b8a99ce..721d5ef 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -2,7 +2,7 @@ "ref\020070727100004.GV27237@ftp.linux.org.uk\0" "From\0Yoann Padioleau <padator@wanadoo.fr>\0" "Subject\0Re: [PATCH 06/68] 0 -> NULL, for arch/frv\0" - "Date\0Fri, 27 Jul 2007 10:21:53 +0000\0" + "Date\0Fri, 27 Jul 2007 12:21:53 +0200\0" "To\0Al Viro <viro@ftp.linux.org.uk>\0" "Cc\0Yoann Padioleau <padator@wanadoo.fr>" kernel-janitors@vger.kernel.org @@ -24,14 +24,14 @@ ">> \n" ">> \t/* allocate some common virtual space to map the new pages */\n" ">> \tarea = get_vm_area(size, VM_ALLOC);\n" - ">> -\tif (area = 0) {\n" - ">> +\tif (area = NULL) {\n" + ">> -\tif (area == 0) {\n" + ">> +\tif (area == NULL) {\n" ">> \t\tfree_pages(page, order);\n" ">> \t\treturn NULL;\n" ">\n" "> Same comment about comparisons with NULL after allocation...\n" "\n" "I don't understand. pte is a pointer right ? So why should we\n" - keep the = 0 ? + keep the == 0 ? -475968b29a0e4fce92ce8ff00fe44702cf75539c23894fcf5fce00c51dba9f04 +fe94792ee917e4f45d38298b7e78a41d95e7664844242c80b16e6f2e6f9030f5
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.