Return-Path: X-Original-To: apmail-hbase-issues-archive@www.apache.org Delivered-To: apmail-hbase-issues-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 195F61098C for ; Sat, 29 Mar 2014 11:14:17 +0000 (UTC) Received: (qmail 12567 invoked by uid 500); 29 Mar 2014 11:14:17 -0000 Delivered-To: apmail-hbase-issues-archive@hbase.apache.org Received: (qmail 12272 invoked by uid 500); 29 Mar 2014 11:14:16 -0000 Mailing-List: contact issues-help@hbase.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Delivered-To: mailing list issues@hbase.apache.org Received: (qmail 12255 invoked by uid 99); 29 Mar 2014 11:14:15 -0000 Received: from arcas.apache.org (HELO arcas.apache.org) (140.211.11.28) by apache.org (qpsmtpd/0.29) with ESMTP; Sat, 29 Mar 2014 11:14:15 +0000 Date: Sat, 29 Mar 2014 11:14:15 +0000 (UTC) From: "Anoop Sam John (JIRA)" To: issues@hbase.apache.org Message-ID: In-Reply-To: References: Subject: [jira] [Updated] (HBASE-10860) Insufficient AccessController covering permission check MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-JIRA-FingerPrint: 30527f35849b9dde25b450d4833f0394 [ https://issues.apache.org/jira/browse/HBASE-10860?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Anoop Sam John updated HBASE-10860: ----------------------------------- Resolution: Fixed Hadoop Flags: Reviewed Status: Resolved (was: Patch Available) Committed to 98 and Trunk. Thanks for the reviews Andy & Ram. > Insufficient AccessController covering permission check > ------------------------------------------------------- > > Key: HBASE-10860 > URL: https://issues.apache.org/jira/browse/HBASE-10860 > Project: HBase > Issue Type: Bug > Components: security > Affects Versions: 0.98.0 > Reporter: Anoop Sam John > Assignee: Anoop Sam John > Fix For: 0.99.0, 0.98.2 > > Attachments: HBASE-10860.patch > > > {code} > List list = (List)entry.getValue(); > if (list == null || list.isEmpty()) { > get.addFamily(col); > } else { > for (Cell cell : list) { > get.addColumn(col, CellUtil.cloneQualifier(cell)); > } > } > {code} > When a delete family Mutation comes, a Cell will be added into the list with Qualifier as null. (See Delete#deleteFamily(byte[])). So it will miss getting added against the check list == null || list.isEmpty(). We will fail getting the cells under this cf for covering permission check. -- This message was sent by Atlassian JIRA (v6.2#6252)