110557 2013-02-21 22:25:43 -0800 DatabaseTracker::getMaxSizeForDatabase() can return an erroneously large value 2013-02-25 15:58:00 -0800 1 1 1 Unclassified WebKit WebCore Misc. 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED InRadar P2 Normal --- 110600 1 mark.lam mark.lam beidson benjamin ggaren michaeln sam oldest_to_newest 839239 0 mark.lam 2013-02-21 22:25:43 -0800 Currently DatabaseTracker::getMaxSizeForDatabase() estimates the max allowed size for a database based on the following formula: maxSize = quota - currentDiskUsage + databaseFileSize; currentDiskUsage is read from the disk. If some bug or external factor should be able to cause currentDiskUsage to be greater than quota + databaseFileSize, then maxSize will be negative and be interpreted as an unreasonably large size that far exceeds the quota. We should fix this by adding some sanity checks to bound the maxSize just in case. 839244 1 189687 mark.lam 2013-02-21 22:32:09 -0800 Created attachment 189687 the fix 839263 2 189687 ggaren 2013-02-21 22:59:03 -0800 Comment on attachment 189687 the fix The comment here says that we're guarding against currentUsage > quota + databaseSize. But the code says we're guarding against currentUsage > quota or databaseFileSize > quota or quota - currentUsage + databaseFileSize > quota. The test case you created showed currentUsage > quota. The most direct way to fix that is: if (currentUsage > quota) currentUsage = quota; With that check, do any of the other checks add anything? 839268 3 mark.lam 2013-02-21 23:06:13 -0800 (In reply to comment #2) > (From update of attachment 189687 [details]) > The comment here says that we're guarding against currentUsage > quota + databaseSize. But the code says we're guarding against currentUsage > quota or databaseFileSize > quota or quota - currentUsage + databaseFileSize > quota. > > The test case you created showed currentUsage > quota. The most direct way to fix that is: > if (currentUsage > quota) > currentUsage = quota; > > With that check, do any of the other checks add anything? I'm being paranoid in case databaseFileSize somehow gets larger than quota as well. So, I cap the maxSize to quota. If that happens I'm telling the database that I want it to be smaller as opposed to artificially endorsing its greater than quota size. Whether the sqlite database can shrink to the smaller size or not is a whole separate issue. With that I can re-write the logic as: if (currentUsage > quota) currentUsage = quota; maxSize = quota - currentUsage + databaseFileSize; if (maxSize > quota) maxSize = quota; If you're ok with that, I can go with that. 839368 4 189687 benjamin 2013-02-22 01:32:11 -0800 Comment on attachment 189687 the fix View in context: https://bugs.webkit.org/attachment.cgi?id=189687&action=review Can't you test this? For example, wouldn't this go to the branch?: 1) Assign a large quota. 2) Create a giant database 3) Reduce the Quota 4) ... 5) Profit! > Source/WebCore/ChangeLog:10 > + > + No new tests. > + Maybe add a word of explanation in the ChangeLog. Why no test? > Source/WebCore/Modules/webdatabase/DatabaseTracker.cpp:312 > + // The quota comes from a configuration value while the currentUsage comes > + // from a measurement of the real disk usage. Let's consider a situation > + // where there is a bug (regardless of the source) where the currentUsage gets > + // unreasonably large such that currentUsage > quota + databaseSize. In that > + // case, maxSize, computed as (quota - currentUsage + databaseFileSize) will > + // be negative and will be misinterpreted as a very large number, thereby > + // effectively granting the database permission to grow unboundedly beyond the > + // quota. > + // > + // Hence, we will do some explicit checks here to guard against this, and > + // ensure some sanity. I am really not a fan of this comment. It reads like you have a mysterious bug elsewhere, and instead of finding it, you just added this workaround here. If you add an explanation here, it should be of the real use case, not a bug (regardless of the source). 839430 5 mark.lam 2013-02-22 03:28:38 -0800 (In reply to comment #4) > (From update of attachment 189687 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=189687&action=review > > Can't you test this? > > For example, wouldn't this go to the branch?: > 1) Assign a large quota. > 2) Create a giant database > 3) Reduce the Quota > 4) ... > 5) Profit! Thanks for your comment. I presume you are hinting at some edge cases where the currentUsage gets so large that it becomes negative for the int64_t values I'm using. I'm going to change the type of all the quantities to uint64_t. With a signed int64_t, there is a theoretical possibility of an overflow of currentUsage and databaseFileSize. In practice, I think that is unrealistic. Regardless, it is a much stronger position to use unsigned values and not have to deal with the overflow conditions. If you are trying to hint at something else other than this, please enlighten me. Thanks. > > Source/WebCore/ChangeLog:10 > > + > > + No new tests. > > + > > Maybe add a word of explanation in the ChangeLog. > > Why no test? No test because this is a trivial change, time is limited, and my time is better spent on fixing the real bug (which is not the purpose of this patch ... see below). > > Source/WebCore/Modules/webdatabase/DatabaseTracker.cpp:312 > > + // The quota comes from a configuration value while the currentUsage comes > > + // from a measurement of the real disk usage. Let's consider a situation > > + // where there is a bug (regardless of the source) where the currentUsage gets > > + // unreasonably large such that currentUsage > quota + databaseSize. In that > > + // case, maxSize, computed as (quota - currentUsage + databaseFileSize) will > > + // be negative and will be misinterpreted as a very large number, thereby > > + // effectively granting the database permission to grow unboundedly beyond the > > + // quota. > > + // > > + // Hence, we will do some explicit checks here to guard against this, and > > + // ensure some sanity. > > I am really not a fan of this comment. It reads like you have a mysterious bug elsewhere, and instead of finding it, you just added this workaround here. > If you add an explanation here, it should be of the real use case, not a bug (regardless of the source). I purposely did not talk about the real bug (not because I didn't know what it is, but) because it is not the intent of this patch to fix the bug. I am working on the fix for the real bug in a separate effort. The intent of this patch is to provide a sanity check that enforces the contract of the function DatabaseTracker::getMaxSizeOfDatabase(). My understanding of the contract is that the size of the database should not be allowed to grow to exceed the quota for its domain origin. Hence, the checks added in my patch ensures that: 1. the estimated allowed growth (i.e. quota - currentUsage) is reasonable (i.e. quota - currentUsage <= quota, i.e. current <= quota). 2. the returned value of "max size of database" is reasonable (i.e. maxSize <= quota). For realistic values of quota, currentUsage, and databaseFileSize, I believe the logic of the patch is sound. A realistic value of currentUsage and databaseSize in this case does not include them being so large that they overflow into negative values (though I've not explained why here). But I agree that it is better to make them unsigned and just avoid even the possibility of complications due to overflow. Note: "quota" is expected to come from a trusted source and is a sane value (hence, my mentioning "configuration value"). "currentUsage" and "databaseFileSize" is not to be trusted. If the damage is already done by someone else to get a currentUsage and/or databaseFileSize that is larger than we should allow, it is not the role of DatabaseTracker::getMaxSizeOfDatabase() to fix that (i.e. reduce it, or prevent it) nor does it have any ability to. The fix for such an issue lies elsewhere. However, it is the role of DatabaseTracker::getMaxSizeOfDatabase() to compute what it considers to be a sane estimate of the max size of the database in question. As for the comment, I intended it to give some context and motivation for why the sanity checks are meaningful. It is not appropriate to document the specific bug I'm guarding against here just because it happens to be the instance that pointed out that we could use some defensive coding here in DatabaseTracker::getMaxSizeOfDatabase(). That said, I could probably write it better so as to not give the wrong impression that I'm just trying to band-aid a bug here. I'll look into how I can do that. 839538 6 mark.lam 2013-02-22 06:13:09 -0800 (In reply to comment #5) > > Why no test? > > No test because this is a trivial change, time is limited, and my time is better spent on fixing the real bug (which is not the purpose of this patch ... see below). Ok, that wasn't an appropriate answer. Let me provide some more details as to why I'm choosing to not implement a layout test that checks for the specific bug that I'm working on fixing: 1. There are already existing tests that check the quota functionality in general: storage/websql/open-database-over-quota.html: tests quota enforcement when opening databases. storage/websql/quota-tracking.html: tests quota enforcement when writing records. However, these only tests quota management for the single process scenario. 2. I could be wrong but I don't think we have a test harness for executing multi-process stress tests yet. That said, while I don't have a layout test to add, I do have a test that I do run by hand to verify the existence of the error condition (with the help of instrumented logging), and then to verify the fix. > I purposely did not talk about the real bug (not because I didn't know what it is, but) because it is not the intent of this patch to fix the bug. I am working on the fix for the real bug in a separate effort. FYI, the "real" bug that highlighted the need for the sanity check in this patch is at https://bugs.webkit.org/show_bug.cgi?id=110600. Again, unlike bug 110600, this bug, 110557, is only intended to put in place a sanity check that enforces the contract of DatabaseTracker::getMaxSizeForDatabase(). It is a defensive maneuver to limit the damage that 110600 (and potentially other bugs like it in the future) can cause. 839683 7 ggaren 2013-02-22 10:00:39 -0800 > I purposely did not talk about the real bug.... I have to agree with Ben: There's no way to understand this code without direct explanation of what it fixes. The bug we're trying to fix is this: If any bug anywhere ever allows an origin to exceed its quota, even by a byte, the origin will thereafter exceed its quota by 2^64. > I'm being paranoid in case databaseFileSize somehow gets larger than quota as well. Paranoia is fear of the unknown. Fear is the mindkiller. Let's not litter our code with fear. Instead, let's document the known things we're trying to fix. Total disk usage for this origin may be greater than quota. Total disk usage is a superset of this individual database's usage, so checking total disk usage is sufficient. How about this: unsigned long long quota = quotaForOriginNoLock(origin); unsigned long long diskUsage = originQuotaManager().diskUsage(origin); unsigned long long databaseFileSize = SQLiteFileSystem::getDatabaseFileSize(database->fileName()); // A previous error may have allowed the database to exceed its quota. Don't multiply that error through // integer underflow, or the quota will permanently become 2^64. if (diskUsage > quota) return 0; ASSERT(databaseFileSize < diskUsage); return quota - diskUsage + databaseFileSize; 839790 8 mark.lam 2013-02-22 11:54:55 -0800 (In reply to comment #7) > if (diskUsage > quota) > return 0; > ASSERT(databaseFileSize < diskUsage); > > return quota - diskUsage + databaseFileSize; That won't work. Here's why: diskUsage comes from OriginUsageManager::diskUsage() which uses a cached value from a stat of the filesystem. The cached value will remain in use as long as the current process hasn't executed a database action that invalidates it. On the other hand, databaseFileSize comes from SQLiteFileSystem::getDatabaseFileSize() which is a stat of the database file every time. So, imagine a scenario where process P1 and P2 both opens a database with the same name. P1 cached the diskUsage at one time and didn't execute any action that invalidates it till now. Meanwhile P2 has been busy adding more data to the database file thereby increasing its size. And then in P1, we get to this code in getMaxSizeForDatabase(). P1 uses its cached diskUsage (which at one time is greater then databaseFileSize) and fetches the current databaseFileSize (which P2 has been pumping up). The result the databaseFileSize can be greater than diskUsage. So, how about we do this instead: unsigned long long quota = quotaForOriginNoLock(origin); unsigned long long diskUsage = originQuotaManager().diskUsage(origin); unsigned long long databaseFileSize = SQLiteFileSystem::getDatabaseFileSize(database->fileName()); // A previous error may have allowed the database to exceed its quota. Don't multiply that error through // integer underflow, or the quota will permanently become 2^64. if (diskUsage > quota) return 0; unsigned long long maxSize = quota - diskUsage + databaseFileSize; if (maxSize > quota) return 0; return maxSize; 839819 9 ggaren 2013-02-22 12:35:59 -0800 > That won't work. Here's why: Great! Now we're making progress with specifics. > So, how about we do this instead: > > unsigned long long quota = quotaForOriginNoLock(origin); > unsigned long long diskUsage = originQuotaManager().diskUsage(origin); > unsigned long long databaseFileSize = SQLiteFileSystem::getDatabaseFileSize(database->fileName()); > > // A previous error may have allowed the database to exceed its quota. Don't multiply that error through > // integer underflow, or the quota will permanently become 2^64. > if (diskUsage > quota) > return 0; > unsigned long long maxSize = quota - diskUsage + databaseFileSize; > if (maxSize > quota) > return 0; > return maxSize; I think this would be slightly clearer like so: // A previous error may have allowed the origin to exceed its quota, or may have allowed this database // to exceed our cached estimate of the origin. Don't multiply that error through integer underflow, or // the quota will permanently become 2^64. unsigned long long maxSize = quota - diskUsage + databaseFileSize; if (maxSize > quota) return 0; return maxSize; I updated the comment to mention the two specific errors we're concerned about, and I changed the code not to test for disUsage specifically, since the only real protection we're providing here is against underflow of maxSize. 839921 10 189821 mark.lam 2013-02-22 14:02:29 -0800 Created attachment 189821 Test for opening a randomly named database and consuming storage space. 839923 11 189822 mark.lam 2013-02-22 14:03:15 -0800 Created attachment 189822 updated fix. 839931 12 189822 ggaren 2013-02-22 14:06:27 -0800 Comment on attachment 189822 updated fix. r=me 839941 13 mark.lam 2013-02-22 14:10:55 -0800 Thanks for the review. Landed in r143791: <http://trac.webkit.org/changeset/143791>. 839953 14 mark.lam 2013-02-22 14:14:34 -0800 <rdar://problem/13260686> 841079 15 mark.lam 2013-02-25 10:15:46 -0800 The value returned by DatabaseTracker::getMaxSizeForDatabase() is used to set the SQLite3 database size using a sql command "PRAGMA max_page_count = <size>”. The SQLite3 documentation on this pragma says, "The maximum page count cannot be reduced below the current database size." It is undefined what will happen if we set the sqlite max_page_count to 0. However, further testing shows that having getMaxSizeForDatabase() return a 0 (when a quota breach is detected), proves to be ineffective at stopping runaway database growth. In contrast, having getMaxSizeForDatabase() return the size of the existing database seems to do the job. In the interest of time, I’m proposing to not dive into the SQLite3 code at this time to figure out the specifics. I believe it is reasonable to have getMaxSizeForDatabase() return the current size when the quota limit has been reached, and testing shows that it works so far. Unless we have more data that say we should do otherwise, I propose going with this solution for the present, and perhaps dig further into the SQLite3 library for details at a later time. Note: here are some testing data points that guided me to this conclusion: When the quota limit has been reached, I tried having getMaxSizeForDatabase() return various values: Value returned Reason for value Result ========= ========== ==== 0 current approach Fails to stop growth. 1024 < sqlite page size Fails to stop growth. 2048 < sqlite page size Fails to stop growth. 2048 < sqlite page size Fails to stop growth. 3072 < sqlite page size Fails to stop growth. 4096 1 page Growth stops. actual DB size return current size Growth stops. SQLiteDatabase::setMaximumSize() calculates the max page count (for the pragma) by dividing the new max database size by the page size. Hence, anything less than 4096 will effectively set a max page count of 0, and that has proven consistently to fail to stop the growth. While returning the page size 4096 seems to work, that seems like a somewhat arbitrary value to have getMaxSizeForDatabase() return. The most meaningful return value would be the current database size. 841081 16 190085 mark.lam 2013-02-25 10:17:12 -0800 Created attachment 190085 Patch for returning current database size instead of 0. 841082 17 190085 mark.lam 2013-02-25 10:19:18 -0800 Comment on attachment 190085 Patch for returning current database size instead of 0. View in context: https://bugs.webkit.org/attachment.cgi?id=190085&action=review > Source/WebCore/ChangeLog:4 > + to return the previous database size instead of 0. That doesn’t read well. I’ll change this to “Changed DatabaseTracker::getMaxSizeForDatabase() to return the previous database size instead of 0 when the quota limit has been reached.” 841102 18 190085 ggaren 2013-02-25 10:42:36 -0800 Comment on attachment 190085 Patch for returning current database size instead of 0. r=me 841150 19 mark.lam 2013-02-25 11:38:56 -0800 Thanks for the review. Landed in r143954: <http://trac.webkit.org/changeset/143954>. 189687 2013-02-21 22:32:09 -0800 2013-02-22 14:03:15 -0800 the fix bug-110557.patch text/plain 2767 mark.lam SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDE0MzY4OSkKKysrIFNvdXJjZS9XZWJDb3JlL0NoYW5n ZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE2IEBACisyMDEzLTAyLTIxICBNYXJrIExh bSAgPG1hcmsubGFtQGFwcGxlLmNvbT4KKworICAgICAgICBBZGQgc29tZSBjaGVja3MgdG8gRGF0 YWJhc2VUcmFja2VyOjpnZXRNYXhTaXplRm9yRGF0YWJhc2UoKSB0byBlbnN1cmUKKyAgICAgICAg dGhhdCBpdCByZXR1cm5zIGEgc2FuZSB2YWx1ZS4KKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtp dC5vcmcvc2hvd19idWcuY2dpP2lkPTExMDU1Ny4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JP RFkgKE9PUFMhKS4KKworICAgICAgICBObyBuZXcgdGVzdHMuCisKKyAgICAgICAgKiBNb2R1bGVz L3dlYmRhdGFiYXNlL0RhdGFiYXNlVHJhY2tlci5jcHA6CisgICAgICAgIChXZWJDb3JlOjpEYXRh YmFzZVRyYWNrZXI6OmdldE1heFNpemVGb3JEYXRhYmFzZSk6CisKIDIwMTMtMDItMjEgIEdyemVn b3J6IEN6YWprb3dza2kgIDxnLmN6YWprb3dza2lAc2Ftc3VuZy5jb20+CiAKICAgICAgICAgQWxs b3cgdG8gcmV0cmlldmUgdGhlIHJlcXVlc3QgZGF0YSBmcm9tIGFic3RyYWN0IFRleHRDaGVja2lu Z1JlcXVlc3QgdG8gYmUgYWNjZXNzaWJsZSBmb3IgV0syCkluZGV4OiBTb3VyY2UvV2ViQ29yZS9N b2R1bGVzL3dlYmRhdGFiYXNlL0RhdGFiYXNlVHJhY2tlci5jcHAKPT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291 cmNlL1dlYkNvcmUvTW9kdWxlcy93ZWJkYXRhYmFzZS9EYXRhYmFzZVRyYWNrZXIuY3BwCShyZXZp c2lvbiAxNDM2ODkpCisrKyBTb3VyY2UvV2ViQ29yZS9Nb2R1bGVzL3dlYmRhdGFiYXNlL0RhdGFi YXNlVHJhY2tlci5jcHAJKHdvcmtpbmcgY29weSkKQEAgLTI5MCw3ICsyOTAsMzcgQEAgdW5zaWdu ZWQgbG9uZyBsb25nIERhdGFiYXNlVHJhY2tlcjo6Z2V0TQogICAgIE11dGV4TG9ja2VyIGxvY2tE YXRhYmFzZShtX2RhdGFiYXNlR3VhcmQpOwogICAgIExvY2tlcjxPcmlnaW5RdW90YU1hbmFnZXI+ IHF1b3RhTWFuYWdlckxvY2tlcihvcmlnaW5RdW90YU1hbmFnZXIoKSk7CiAgICAgU2VjdXJpdHlP cmlnaW4qIG9yaWdpbiA9IGRhdGFiYXNlLT5zZWN1cml0eU9yaWdpbigpOwotICAgIHJldHVybiBx dW90YUZvck9yaWdpbk5vTG9jayhvcmlnaW4pIC0gb3JpZ2luUXVvdGFNYW5hZ2VyKCkuZGlza1Vz YWdlKG9yaWdpbikgKyBTUUxpdGVGaWxlU3lzdGVtOjpnZXREYXRhYmFzZUZpbGVTaXplKGRhdGFi YXNlLT5maWxlTmFtZSgpKTsKKworICAgIGludDY0X3QgcXVvdGEgPSBxdW90YUZvck9yaWdpbk5v TG9jayhvcmlnaW4pOworICAgIGludDY0X3QgY3VycmVudFVzYWdlID0gb3JpZ2luUXVvdGFNYW5h Z2VyKCkuZGlza1VzYWdlKG9yaWdpbik7CisgICAgaW50NjRfdCBkYXRhYmFzZUZpbGVTaXplID0g U1FMaXRlRmlsZVN5c3RlbTo6Z2V0RGF0YWJhc2VGaWxlU2l6ZShkYXRhYmFzZS0+ZmlsZU5hbWUo KSk7CisKKyAgICBBU1NFUlQocXVvdGEgPj0gMCk7CisgICAgQVNTRVJUKGN1cnJlbnRVc2FnZSA+ PSAwKTsKKyAgICBBU1NFUlQoZGF0YWJhc2VGaWxlU2l6ZSA+PSAwKTsKKworICAgIC8vIFRoZSBx dW90YSBjb21lcyBmcm9tIGEgY29uZmlndXJhdGlvbiB2YWx1ZSB3aGlsZSB0aGUgY3VycmVudFVz YWdlIGNvbWVzCisgICAgLy8gZnJvbSBhIG1lYXN1cmVtZW50IG9mIHRoZSByZWFsIGRpc2sgdXNh Z2UuIExldCdzIGNvbnNpZGVyIGEgc2l0dWF0aW9uCisgICAgLy8gd2hlcmUgdGhlcmUgaXMgYSBi dWcgKHJlZ2FyZGxlc3Mgb2YgdGhlIHNvdXJjZSkgd2hlcmUgdGhlIGN1cnJlbnRVc2FnZSBnZXRz CisgICAgLy8gdW5yZWFzb25hYmx5IGxhcmdlIHN1Y2ggdGhhdCBjdXJyZW50VXNhZ2UgPiBxdW90 YSArIGRhdGFiYXNlU2l6ZS4gSW4gdGhhdAorICAgIC8vIGNhc2UsIG1heFNpemUsIGNvbXB1dGVk IGFzIChxdW90YSAtIGN1cnJlbnRVc2FnZSArIGRhdGFiYXNlRmlsZVNpemUpIHdpbGwKKyAgICAv LyBiZSBuZWdhdGl2ZSBhbmQgd2lsbCBiZSBtaXNpbnRlcnByZXRlZCBhcyBhIHZlcnkgbGFyZ2Ug bnVtYmVyLCB0aGVyZWJ5CisgICAgLy8gZWZmZWN0aXZlbHkgZ3JhbnRpbmcgdGhlIGRhdGFiYXNl IHBlcm1pc3Npb24gdG8gZ3JvdyB1bmJvdW5kZWRseSBiZXlvbmQgdGhlCisgICAgLy8gcXVvdGEu CisgICAgLy8KKyAgICAvLyBIZW5jZSwgd2Ugd2lsbCBkbyBzb21lIGV4cGxpY2l0IGNoZWNrcyBo ZXJlIHRvIGd1YXJkIGFnYWluc3QgdGhpcywgYW5kCisgICAgLy8gZW5zdXJlIHNvbWUgc2FuaXR5 LgorCisgICAgaW50NjRfdCBtYXhTaXplOworICAgIGlmIChjdXJyZW50VXNhZ2UgPiBxdW90YSkK KyAgICAgICAgbWF4U2l6ZSA9IGRhdGFiYXNlRmlsZVNpemU7CisgICAgZWxzZQorICAgICAgICBt YXhTaXplID0gcXVvdGEgLSBjdXJyZW50VXNhZ2UgKyBkYXRhYmFzZUZpbGVTaXplOworCisgICAg aWYgKG1heFNpemUgPiBxdW90YSkKKyAgICAgICAgbWF4U2l6ZSA9IHF1b3RhOworCisgICAgcmV0 dXJuIG1heFNpemU7CiB9CiAKIHZvaWQgRGF0YWJhc2VUcmFja2VyOjpkYXRhYmFzZUNoYW5nZWQo RGF0YWJhc2VCYWNrZW5kQmFzZSogZGF0YWJhc2UpCg== 189821 2013-02-22 14:02:29 -0800 2013-02-22 14:02:29 -0800 Test for opening a randomly named database and consuming storage space. quota-test.html text/html 10281 mark.lam PGJvZHk+CjxzY3JpcHQ+Ci8vIFRoaXMgdGVzdCBjcmVhdGVzIGEgZGF0YWJhc2Ugd2l0aCBhIHJh bmRvbSBuYW1lIChzZWUgJ2RiTmFtZScpIGFuZCBhdHRlbXB0cyB0bwovLyBob2cgYXMgbXVjaCBz dG9yYWdlIHNwYWNlIGFzIHBvc3NpYmxlLiBUbyB0ZXN0IGl0IGluIGEgbXVsdGktcHRvY2VzcyBl bnZpcm9ubWVudCwKLy8gbG9hZCB0aGUgcGFnZSBpbiBtdWx0aXBsZSBwcm9jZXNzZXMsIGFuZCBt b25pdG9yIHRoZSBzaXplIG9mIHRoZSBkYXRhYmFzZQovLyBkaXJlY3RvcnkgYW5kIGl0cyBmaWxl cy4KCnZhciBpc0Fib3J0ZWQgPSBmYWxzZTsKCnZhciByYW5kID0gZnVuY3Rpb24obGltaXQpIHsK ICAgIHJldHVybiBNYXRoLnJvdW5kKE1hdGgucmFuZG9tKCkgKiBsaW1pdCk7Cn0KCnZhciByYW5k b21seVdhc3RlVGltZSA9IGZ1bmN0aW9uKCkgewogICAgdmFyIG5lZWRUb1dhaXQgPSAocmFuZCgx MCkgPCAzKTsKICAgIGlmIChuZWVkVG9XYWl0KSB7CiAgICAgICAgdmFyIHN0YXJ0VGltZSA9IERh dGUubm93KCk7CiAgICAgICAgd2hpbGUgKERhdGUubm93KCkgLSBzdGFydFRpbWUgPCAxMDAwKSB7 IAogICAgICAgIH0gLy8gV2FpdCBmb3IgMSBtaW51dGUuCiAgICB9Cn0KCgp2YXIgaW5pdFN0dWZm ID0gZnVuY3Rpb24oZGIpIHsKCiAgICB2YXIgaW5pdFRyYW5zYWN0aW9uRXJyb3JDYWxsYmFjayA9 IGZ1bmN0aW9uKGVycm9yKSB7CiAgICAgICAgYWxlcnQoImVycm9yIHdoaWxlIElOSVQgdGFibGU6 IGVyciA9ICIgKyBlcnJvci5jb2RlICsgIiAnIiArIGVycm9yLm1lc3NhZ2UgKyAiJyIpOwogICAg ICAgIGlzQWJvcnRlZCA9IHRydWU7CiAgICB9CgogICAgdmFyIGluaXRUcmFuc2FjdGlvblN1Y2Nl c3NDYWxsYmFjayA9IGZ1bmN0aW9uKCkgewogICAgICAgIC8vIGFsZXJ0KCJJTklUIHRhYmxlIHN1 Y2NlZWRlZCIpOwogICAgfQoKICAgIHZhciBpbml0U3RhdGVtZW50U3VjY2Vzc0NhbGxiYWNrID0g ZnVuY3Rpb24odHJhbnNhY3Rpb24sIHJlc3VsdHMpIHsKICAgICAgICAvLyBhbGVydCgiU1VDQ0VT UyBpbiBJTklUIHN0YXRlbWVudCIpOwogICAgfQoKICAgIHZhciBpbml0U3RhdGVtZW50RXJyb3JD YWxsYmFjayA9IGZ1bmN0aW9uKHRyYW5zYWN0aW9uLCBlcnJvcikgewogICAgICAgIC8vYWxlcnQo Ik9oIG5vZXMhIFRoZXJlIGhheiBiaW4gYSBkYXRhYmFzZSBlcnJvciBpbiBJTklUIHN0YXRlbWVu dCEgOiBlcnIgPSAiICsgZXJyb3IuY29kZSArICIgJyIgKyBlcnJvci5tZXNzYWdlICsgIiciKTsK ICAgICAgICBpc0Fib3J0ZWQgPSB0cnVlOwogICAgfQoKICAgIC8vIENyZWF0ZSB0aGUgdGFibGU6 CiAgICBkYi50cmFuc2FjdGlvbigKICAgICAgICBmdW5jdGlvbih0cmFuc2FjdGlvbikgewogICAg ICAgICAgICB0cmFuc2FjdGlvbi5leGVjdXRlU3FsKCJDUkVBVEUgVEFCTEUgSUYgTk9UIEVYSVNU UyBzdHVmZiAoIiArCiAgICAgICAgICAgICAgICAiaWQgSU5URUdFUiBOT1QgTlVMTCBQUklNQVJZ IEtFWSBBVVRPSU5DUkVNRU5ULCIgKwogICAgICAgICAgICAgICAgIm5hbWUgVEVYVCBOT1QgTlVM TCwiICsKICAgICAgICAgICAgICAgICJ4IElOVEVHRVIgTk9UIE5VTEwsIiArCiAgICAgICAgICAg ICAgICAieSBJTlRFR0VSIE5PVCBOVUxMLCIgKwogICAgICAgICAgICAgICAgInogSU5URUdFUiBO T1QgTlVMTCwiICsKICAgICAgICAgICAgICAgICJjaGVja3N1bSBJTlRFR0VSIE5PVCBOVUxMKTsi LAogICAgICAgICAgICAgICAgW10sCiAgICAgICAgICAgICAgICBpbml0U3RhdGVtZW50U3VjY2Vz c0NhbGxiYWNrLAogICAgICAgICAgICAgICAgaW5pdFN0YXRlbWVudEVycm9yQ2FsbGJhY2spOwog ICAgICAgIH0sCiAgICAgICAgaW5pdFRyYW5zYWN0aW9uRXJyb3JDYWxsYmFjaywKICAgICAgICBp bml0VHJhbnNhY3Rpb25TdWNjZXNzQ2FsbGJhY2sKICAgICk7Cn07CgovLz09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT0KLy8gSU5TRVJUCgp2YXIgaW5zZXJ0SW50ZXJ2YWw7Cgp2YXIgaW5zZXJ0U3R1ZmYgPSBm dW5jdGlvbihuYW1lLCB4LCB5LCB6LCBjaGVja3N1bSkgewoKICAgIHZhciBpbnNlcnRUcmFuc2Fj dGlvbkVycm9yQ2FsbGJhY2sgPSBmdW5jdGlvbihlcnJvcikgewogICAgICAgIGFsZXJ0KCJlcnJv ciB3aGlsZSBJTlNFUlQgdGFibGU6IGVyciA9ICIgKyBlcnJvci5jb2RlICsgIiAnIiArIGVycm9y Lm1lc3NhZ2UgKyAiJyIpOwogICAgICAgIGNsZWFySW50ZXJ2YWwoaW5zZXJ0SW50ZXJ2YWwpOwog ICAgICAgIGNsZWFySW50ZXJ2YWwobG9hZEludGVydmFsKTsKICAgICAgICBpc0Fib3J0ZWQgPSB0 cnVlOwogICAgfQoKICAgIHZhciBpbnNlcnRUcmFuc2FjdGlvblN1Y2Nlc3NDYWxsYmFjayA9IGZ1 bmN0aW9uKCkgewogICAgICAgIC8vIGFsZXJ0KCJJTlNFUlQgdGFibGUgc3VjY2VlZGVkIik7CiAg ICB9CgogICAgdmFyIGluc2VydFN0YXRlbWVudFN1Y2Nlc3NDYWxsYmFjayA9IGZ1bmN0aW9uKHRy YW5zYWN0aW9uLCByZXN1bHRzKSB7CiAgICAgICAgLy8gYWxlcnQoIlNVQ0NFU1MgaW4gSU5TRVJU IHN0YXRlbWVudCIpOwogICAgfQoKICAgIHZhciBpbnNlcnRTdGF0ZW1lbnRFcnJvckNhbGxiYWNr ID0gZnVuY3Rpb24odHJhbnNhY3Rpb24sIGVycm9yKSB7CiAgICAgICAgYWxlcnQoIk9oIG5vZXMh IFRoZXJlIGhheiBiaW4gYSBkYXRhYmFzZSBlcnJvciBpbiBJTlNFUlQgc3RhdGVtZW50ISA6IGVy ciA9ICIgKyBlcnJvci5jb2RlICsgIiAnIiArIGVycm9yLm1lc3NhZ2UgKyAiJyIpOwogICAgICAg IGNsZWFySW50ZXJ2YWwoaW5zZXJ0SW50ZXJ2YWwpOwogICAgICAgIGNsZWFySW50ZXJ2YWwobG9h ZEludGVydmFsKTsKICAgICAgICBpc0Fib3J0ZWQgPSB0cnVlOwogICAgICAgIHJldHVybiBmYWxz ZTsKICAgIH0KCiAgICBkYi50cmFuc2FjdGlvbigKICAgICAgICBmdW5jdGlvbih0cmFuc2FjdGlv bikgewogICAgICAgICAgICB2YXIgajsKICAgICAgICAgICAgZm9yIChqID0gMDsgaiA8IDEwMDAg JiYgIWlzQWJvcnRlZDsgaisrKSB7CiAgICAgICAgICAgIHRyYW5zYWN0aW9uLmV4ZWN1dGVTcWwo KCJJTlNFUlQgSU5UTyBzdHVmZiAobmFtZSwgeCwgeSwgeiwgY2hlY2tzdW0pIFZBTFVFUyAoPywg PywgPywgPywgPyk7IiksCiAgICAgICAgICAgICAgICBbbmFtZSwgeCwgeSwgeiwgY2hlY2tzdW1d LAogICAgICAgICAgICAgICAgaW5zZXJ0U3RhdGVtZW50U3VjY2Vzc0NhbGxiYWNrLAogICAgICAg ICAgICAgICAgaW5zZXJ0U3RhdGVtZW50RXJyb3JDYWxsYmFjayk7CiAgICAgICAgICAgIH0KICAg ICAgICB9LAogICAgICAgIGluc2VydFRyYW5zYWN0aW9uRXJyb3JDYWxsYmFjaywKICAgICAgICBp bnNlcnRUcmFuc2FjdGlvblN1Y2Nlc3NDYWxsYmFjawogICAgKTsKfQoKLy89PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09Ci8vIFVQREFURQoKdmFyIHVwZGF0ZUNvdW50ID0gMDsKdmFyIHVwZGF0ZUVycm9yQ291 bnQgPSAwOwp2YXIgdXBkYXRlU3VjY2Vzc0NvdW50ID0gMDsKCnZhciB1cGRhdGVTdHVmZiA9IGZ1 bmN0aW9uKG5hbWUsIHgsIHksIHosIGNoZWNrc3VtLCBzdWNjZXNzQ2FsbGJhY2spIHsKICAgIHZh ciBlcnJvcnNBbGxvd2VkID0gNTsKCiAgICB2YXIgdXBkYXRlU3RhdGVtZW50RXJyb3JDYWxsYmFj ayA9IGZ1bmN0aW9uKHRyYW5zYWN0aW9uLCBlcnJvcikgewogICAgICAgIC8vYWxlcnQoIk9oIG5v ZXMhIFRoZXJlIGhheiBiaW4gYSBkYXRhYmFzZSBlcnJvciBpbiBVUERBVEUgc3RhdGVtZW50ISA6 IGVyciA9ICIgKyBlcnJvci5jb2RlICsgIiAnIiArIGVycm9yLm1lc3NhZ2UgKyAiJyIpOwogICAg ICAgIGNvbnNvbGUubG9nKCIgdXBkYXRlU3RhdGVtZW50RXJyb3JDYWxsYmFjayIpOwogICAgICAg IGVycm9yc0FsbG93ZWQtLTsKICAgICAgICBpZiAoZXJyb3JzQWxsb3dlZCA8PSAwKQogICAgICAg ICAgICBpc0Fib3J0ZWQgPSB0cnVlOwogICAgICAgIHJldHVybiBmYWxzZTsKICAgIH0KCiAgICBm dW5jdGlvbiB1cGRhdGVUcmFuc2FjdGlvbkVycm9yQ2FsbGJhY2soZXJyb3IpIHsKICAgICAgICB1 cGRhdGVUcmFuc2FjdGlvbkVycm9yQ291bnQrKzsKICAgICAgICAvL2FsZXJ0KCJPaCBub2VzISBU aGVyZSBoYXogYmluIGEgZGF0YWJhc2UgZXJyb3Igd2hpbGUgU0FWSU5HISBlcnIgPSAiICsgZXJy b3IuY29kZSArICIgJyIgKyBlcnJvci5tZXNzYWdlICsgIiciKTsKICAgICAgICBjb25zb2xlLmxv ZygiIHVwZGF0ZVRyYW5zYWN0aW9uRXJyb3JDYWxsYmFjayIpOwogICAgICAgIGVycm9yc0FsbG93 ZWQtLTsKICAgICAgICBpZiAoZXJyb3JzQWxsb3dlZCA8PSAwKQogICAgICAgICAgICBpc0Fib3J0 ZWQgPSB0cnVlOwogICAgfQoKICAgIGZ1bmN0aW9uIHVwZGF0ZVRyYW5zYWN0aW9uU3VjY2Vzc0Nh bGxiYWNrKCkgewogICAgICAgIHVwZGF0ZVRyYW5zYWN0aW9uU3VjY2Vzc0NvdW50Kys7CiAgICAg ICAgY29uc29sZS5sb2coIiB1cGRhdGVUcmFuc2FjdGlvblN1Y2Nlc3NDYWxsYmFjayIpOwogICAg ICAgIC8vIHJhbmRvbWx5V2FzdGVUaW1lKCk7CiAgICB9CgogICAgdXBkYXRlQ291bnQrKzsKICAg IGRiLnRyYW5zYWN0aW9uKAogICAgICAgIGZ1bmN0aW9uKHRyYW5zYWN0aW9uKSB7CiAgICAgICAg ICAgIGNvbnNvbGUubG9nKCIgU3RhcnQgdHJhbnNhY3Rpb24iKTsKICAgICAgICAgICAgdHJhbnNh Y3Rpb24uZXhlY3V0ZVNxbCgoIlVQREFURSBzdHVmZiBTRVQgeD0/LCB5PT8sIHo9PywgY2hlY2tz dW09PyBXSEVSRSBuYW1lPT87IiksCiAgICAgICAgICAgICAgICBbeCwgeSwgeiwgY2hlY2tzdW0s IG5hbWVdLAogICAgICAgICAgICAgICAgZnVuY3Rpb24odHJhbnNhY3Rpb24sIHJlc3VsdHMpIHsg c3VjY2Vzc0NhbGxiYWNrKHJlc3VsdHMpOyB9LAogICAgICAgICAgICAgICAgdXBkYXRlU3RhdGVt ZW50RXJyb3JDYWxsYmFjayk7CiAgICAgICAgICAgIC8vIHJhbmRvbWx5V2FzdGVUaW1lKCk7CiAg ICAgICAgfSwKICAgICAgICB1cGRhdGVUcmFuc2FjdGlvbkVycm9yQ2FsbGJhY2ssCiAgICAgICAg dXBkYXRlVHJhbnNhY3Rpb25TdWNjZXNzQ2FsbGJhY2sKICAgICk7Cn07CgoKLy89PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09Ci8vIFNFTEVDVAoKdmFyIGxvYWRJbnRlcnZhbDsKCnZhciBsb2FkU3R1ZmYgPSBm dW5jdGlvbihuYW1lLCBzdWNjZXNzQ2FsbGJhY2spIHsKICAgIHZhciBlcnJvcnNBbGxvd2VkID0g NTsKICAgIHZhciBsb2FkU3RhdGVtZW50RXJyb3JDYWxsYmFjayA9IGZ1bmN0aW9uKHRyYW5zYWN0 aW9uLCBlcnJvcikgewogICAgICAgIC8vYWxlcnQoIk9oIG5vZXMhIFRoZXJlIGhheiBiaW4gYSBk YXRhYmFzZSBlcnJvciBpbiBMT0FEIHN0YXRlbWVudCEgZXJyID0gIiArIGVycm9yLmNvZGUgKyAi ICciICsgZXJyb3IubWVzc2FnZSArICInIik7CiAgICAgICAgY29uc29sZS5sb2coIiBsb2FkU3Rh dGVtZW50RXJyb3JDYWxsYmFjayIpOwogICAgICAgIGVycm9yc0FsbG93ZWQtLTsKICAgICAgICBp ZiAoZXJyb3JzQWxsb3dlZCA8PSAwKSB7CiAgICAgICAgICAgIGNsZWFySW50ZXJ2YWwobG9hZElu dGVydmFsKTsKICAgICAgICAgICAgaXNBYm9ydGVkID0gdHJ1ZTsKICAgICAgICB9CiAgICAgICAg cmV0dXJuIGZhbHNlOwogICAgfQoKCiAgICBmdW5jdGlvbiBsb2FkVHJhbnNhY3Rpb25FcnJvckNh bGxiYWNrKGVycm9yKSB7CiAgICAgICAgLy9hbGVydCgiT2ggbm9lcyEgVGhlcmUgaGF6IGJpbiBh IGRhdGFiYXNlIGVycm9yIHdoaWxlIExPQURJTkchIGVyciA9ICIgKyBlcnJvci5jb2RlICsgIiAn IiArIGVycm9yLm1lc3NhZ2UgKyAiJyIpOwogICAgICAgIGNvbnNvbGUubG9nKCIgbG9hZFRyYW5z YWN0aW9uRXJyb3JDYWxsYmFjayIpOwogICAgICAgIGVycm9yc0FsbG93ZWQtLTsKICAgICAgICBp ZiAoZXJyb3JzQWxsb3dlZCA8PSAwKSB7CiAgICAgICAgICAgIGNsZWFySW50ZXJ2YWwobG9hZElu dGVydmFsKTsKICAgICAgICAgICAgaXNBYm9ydGVkID0gdHJ1ZTsKICAgICAgICB9CiAgICB9CiAg ICBmdW5jdGlvbiBsb2FkVHJhbnNhY3Rpb25TdWNjZXNzQ2FsbGJhY2soKSB7CiAgICAgICAgY29u c29sZS5sb2coIiBsb2FkVHJhbnNhY3Rpb25TdWNjZXNzQ2FsbGJhY2siKTsKICAgICAgICAvLyBy YW5kb21seVdhc3RlVGltZSgpOwogICAgfQoKICAgIGRiLnJlYWRUcmFuc2FjdGlvbigKICAgICAg ICBmdW5jdGlvbih0cmFuc2FjdGlvbikgewogICAgICAgICAgICB0cmFuc2FjdGlvbi5leGVjdXRl U3FsKCgiU0VMRUNUICogRlJPTSBzdHVmZiBXSEVSRSBuYW1lPT8iKSwKICAgICAgICAgICAgICAg IFtuYW1lXSwKICAgICAgICAgICAgICAgIGZ1bmN0aW9uKHRyYW5zYWN0aW9uLCByZXN1bHRzKSB7 IHN1Y2Nlc3NDYWxsYmFjayhyZXN1bHRzKTsgfSwKICAgICAgICAgICAgICAgIGxvYWRTdGF0ZW1l bnRFcnJvckNhbGxiYWNrKTsKICAgICAgICAgICAgLy8gcmFuZG9tbHlXYXN0ZVRpbWUoKTsKICAg ICAgICAgfSwKICAgICAgICAgbG9hZFRyYW5zYWN0aW9uRXJyb3JDYWxsYmFjaywKICAgICAgICAg bG9hZFRyYW5zYWN0aW9uU3VjY2Vzc0NhbGxiYWNrCiAgICApOwp9OwoKLy89PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09Cgp2YXIgY3JlYXRpb25DYWxsYmFjayA9IGZ1bmN0aW9uKCkgewogICAgLy8gYWxlcnQo IkRhdGFiYXNlIG9wZW5lZCEiKTsKfQoKLy8gT3BlbiB0aGUgZGF0YWJhc2U6CnZhciBkYk5hbWUg PSAncXVvdGEnICsgcmFuZCgxMCk7Ci8vIHZhciBkYk5hbWUgPSAncXVvdGEnOwp2YXIgZGIgPSBv cGVuRGF0YWJhc2UoZGJOYW1lLCAnMS4wJywgJ0NvcnJ1cHQgTWUgUGxlYXNlJywgMzIqMTAyNCwg Y3JlYXRpb25DYWxsYmFjayk7Cgppbml0U3R1ZmYoZGIpOwoKd2luZG93Lm9ubG9hZCA9IGZ1bmN0 aW9uKGUpIHsKICAgIC8vIExldCB0aGUgbWF5aGVtIGJlZ2luOgoKICAgIHZhciBudW1FbnRyaWVz ID0gNTsKICAgIHZhciByYW5kTGltaXQgPSAxMDA7CiAgICB2YXIgaTsKCiAgICBmb3IgKGkgPSAx OyBpIDw9IG51bUVudHJpZXMgJiYgIWlzQWJvcnRlZDsgaSsrKSB7CiAgICAgICAgdmFyIG5hbWUg PSAiZW50cnkiICsgaTsKICAgICAgICB2YXIgeCA9IHJhbmQocmFuZExpbWl0KTsKICAgICAgICB2 YXIgeSA9IHJhbmQocmFuZExpbWl0KTsKICAgICAgICB2YXIgeiA9IHJhbmQocmFuZExpbWl0KTsK ICAgICAgICB2YXIgY2hlY2tzdW0gPSB4ICsgeSArIHo7CiAgICAgICAgbG9hZFN0dWZmKG5hbWUs IGZ1bmN0aW9uKHJlc3VsdHMpIHsKICAgICAgICAgICAgLy8gYWxlcnQoIlsiICsgaSArICJdIG5h bWUgJyIgKyBuYW1lICsgIicgcmVzdWx0cy5yb3dzLmxlbmd0aCA9ICIgKyByZXN1bHRzLnJvd3Mu bGVuZ3RoKTsKICAgICAgICAgICAgaWYgKHJlc3VsdHMucm93cy5sZW5ndGggPT0gMCkgewogICAg ICAgICAgICAgICAgLy8gRW50cnkgbm90IGZvdW5kLiAgTWFrZSBvbmU6CiAgICAgICAgICAgICAg ICAvLyBpbnNlcnRTdHVmZihuYW1lLCB4LCB5LCB6LCBjaGVja3N1bSk7CiAgICAgICAgICAgIH0g ICAgICAgIAogICAgICAgIH0pOwogICAgICAgIGluc2VydFN0dWZmKG5hbWUsIHgsIHksIHosIGNo ZWNrc3VtKTsKICAgIH0KCiAgICB2YXIgbmFtZTsKICAgIHZhciB4OwogICAgdmFyIHk7CiAgICB2 YXIgejsKICAgIHZhciBjaGVja3N1bTsKICAgIHZhciBpOwogICAgdmFyIHJlc3VsdFN0cjsKICAg IHZhciByb3c7CgoKICAgIHN0YXJ0TWF5aGVtID0gZnVuY3Rpb24oKSB7CiAgICAgICAgZnVuY3Rp b24gdmVyaWZ5SW5zZXJ0KHJlc3VsdHMpIHsKICAgICAgICAgICAgLy92YXIgcmVzdWx0c1N0ciA9 ICJJTlNFUlQgcmVzdWx0cy5yb3dzLmxlbmd0aCA9ICIgKyByZXN1bHRzLnJvd3MubGVuZ3RoICsg IlxuIjsKICAgICAgICAgICAgLy9hbGVydChyZXN1bHRzU3RyKTsKICAgICAgICB9CiAgICAgICAg aW5zZXJ0SW50ZXJ2YWwgPSBzZXRJbnRlcnZhbChmdW5jdGlvbigpIHsKICAgICAgICAgICAgZm9y IChpID0gMTsgaSA8PSAxMDAwICYmICFpc0Fib3J0ZWQ7IGkrKykgewogICAgICAgICAgICAgICAg bmFtZSA9ICJlbnRyeSIgKyByYW5kKDEwKTsKICAgICAgICAgICAgICAgIHggPSByYW5kKHJhbmRM aW1pdCk7CiAgICAgICAgICAgICAgICB5ID0gcmFuZChyYW5kTGltaXQpOwogICAgICAgICAgICAg ICAgeiA9IHJhbmQocmFuZExpbWl0KTsKICAgICAgICAgICAgICAgIGNoZWNrc3VtID0geCArIHkg KyB6OwogICAgICAgICAgICAgICAgaW5zZXJ0U3R1ZmYobmFtZSwgeCwgeSwgeiwgY2hlY2tzdW0s IHZlcmlmeUluc2VydCk7CiAgICAgICAgICAgICB9CgogICAgICAgIH0sIDMzKTsKCiAgICAgICAg dmFyIGxvYWRDb3VudCA9IDA7CiAgICAgICAgZnVuY3Rpb24gdmVyaWZ5TG9hZChyZXN1bHRzKSB7 CiAgICAgICAgICAgIGxvYWRDb3VudCsrOwogICAgICAgICAgICBpZiAobG9hZENvdW50ID4gMTAw KSBsb2FkQ291bnQgPSAwOwoKICAgICAgICAgICAgdmFyIHN0YXJ0SW5kZXggPSAwOwogICAgICAg ICAgICBpZiAocmVzdWx0cy5yb3dzLmxlbmd0aCA+IDUpIHsKICAgICAgICAgICAgICAgIHJlc3Vs dFN0ciArPSAiLi4uXG4iOwogICAgICAgICAgICAgICAgc3RhcnRJbmRleCA9IHJlc3VsdHMucm93 cy5sZW5ndGggLSA1OwogICAgICAgICAgICB9CiAgICAgICAgICAgIGZvciAoaSA9IHN0YXJ0SW5k ZXg7IGkgPCByZXN1bHRzLnJvd3MubGVuZ3RoICYmICFpc0Fib3J0ZWQ7IGkrKykgewogICAgICAg ICAgICAgICAgcm93ID0gcmVzdWx0cy5yb3dzLml0ZW0oaSk7CiAgICAgICAgICAgICAgICBjaGVj a3N1bSA9IHJvdy54ICsgcm93LnkgKyByb3cuejsKICAgICAgICAgICAgICAgIHZhciBtYXRjaCA9 IChjaGVja3N1bSA9PT0gcm93LmNoZWNrc3VtKTsKCiAgICAgICAgICAgICAgICBpZiAobG9hZENv dW50ID09IDApIHsKICAgICAgICAgICAgICAgICAgICByZXN1bHRTdHIgKz0gcm93Lm5hbWUgKyAi OiAiICsgcm93LnggKyAiICsgIiArIHJvdy55ICsgIiArICIgKyByb3cueiArICIgPT0gIiArIHJv dy5jaGVja3N1bTsKICAgICAgICAgICAgICAgICAgICBpZiAoIW1hdGNoKQogICAgICAgICAgICAg ICAgICAgICAgICByZXN1bHRTdHIgKz0gIiB8IEZBSUxFRCA9ICIgKyBjaGVja3N1bTsKICAgICAg ICAgICAgICAgICAgICByZXN1bHRTdHIgKz0gIlxuIjsKICAgICAgICAgICAgICAgICAgICBpZiAo IW1hdGNoKQogICAgICAgICAgICAgICAgICAgICAgICBhbGVydChyZXN1bHRTdHIpOwogICAgICAg ICAgICAgICAgICAgIGlmIChyb3cubmFtZSA9PSAoImVudHJ5IiArIG51bUVudHJpZXMpKSB7CiAg ICAgICAgICAgICAgICAgICAgICAgIGRvY3VtZW50LmJvZHkuaW5uZXJUZXh0ID0gcmVzdWx0U3Ry OwogICAgICAgICAgICAgICAgICAgICAgICByZXN1bHRTdHIgPSAiTE9BRCByZXN1bHRzOlxuIjsK ICAgICAgICAgICAgICAgICAgICB9CiAgICAgICAgICAgICAgICB9IGVsc2UgewogICAgICAgICAg ICAgICAgICAgIGlmICghbWF0Y2gpIHsKICAgICAgICAgICAgICAgICAgICAgICAgcmVzdWx0U3Ry ICs9IHJvdy5uYW1lICsgIjogIiArIHJvdy54ICsgIiArICIgKyByb3cueSArICIgKyAiICsgcm93 LnogKyAiID09ICIgKyByb3cuY2hlY2tzdW07CiAgICAgICAgICAgICAgICAgICAgICAgIHJlc3Vs dFN0ciArPSAiIHwgRkFJTEVEID0gIiArIGNoZWNrc3VtOwogICAgICAgICAgICAgICAgICAgICAg ICByZXN1bHRTdHIgKz0gIlxuIjsKICAgICAgICAgICAgICAgICAgICB9CiAgICAgICAgICAgICAg ICB9CiAgICAgICAgICAgIH0KICAgICAgICAgICAgLy8gcmFuZG9tbHlXYXN0ZVRpbWUoKTsKICAg ICAgICB9CgogICAgICAgIHJlc3VsdFN0ciA9ICJMT0FEIHJlc3VsdHM6XG4iOwogICAgICAgIGxv YWRJbnRlcnZhbCA9IHNldEludGVydmFsKGZ1bmN0aW9uKCkgewogICAgICAgICAgICBmb3IgKGkg PSAxOyBpIDw9IG51bUVudHJpZXMgJiYgIWlzQWJvcnRlZDsgaSsrKSB7CiAgICAgICAgICAgICAg ICBuYW1lID0gImVudHJ5IiArIGk7CiAgICAgICAgICAgICAgICBsb2FkU3R1ZmYobmFtZSwgdmVy aWZ5TG9hZCk7CiAgICAgICAgICAgICB9CiAgICAgICAgfSwgMTAwMCk7CiAgICB9CgogICAgaWYg KCFpc0Fib3J0ZWQpCiAgICAgICAgc2V0VGltZW91dCgic3RhcnRNYXloZW0oKSIsIDUwMDApOwp9 OwoKPC9zY3JpcHQ+CjwvYm9keT4K 189822 2013-02-22 14:03:15 -0800 2013-02-22 14:06:27 -0800 updated fix. bug-110557.patch text/plain 2585 mark.lam SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDE0Mzc4OCkKKysrIFNvdXJjZS9XZWJDb3JlL0NoYW5n ZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDIxIEBACisyMDEzLTAyLTIxICBNYXJrIExh bSAgPG1hcmsubGFtQGFwcGxlLmNvbT4KKworICAgICAgICBBZGQgc29tZSBjaGVja3MgdG8gRGF0 YWJhc2VUcmFja2VyOjpnZXRNYXhTaXplRm9yRGF0YWJhc2UoKSB0byBlbnN1cmUKKyAgICAgICAg dGhhdCBpdCByZXR1cm5zIGEgc2FuZSB2YWx1ZS4KKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtp dC5vcmcvc2hvd19idWcuY2dpP2lkPTExMDU1Ny4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JP RFkgKE9PUFMhKS4KKworICAgICAgICBObyBsYXlvdXQgdGVzdCwgYnV0IHRoZXJlIGlzIGEgcXVv dGEtdGVzdC5odG1sIGF0dGFjaGVkIHRvIGJ1Z3ppbGxhLgorICAgICAgICBUaGUgdGVzdCBpcyBh IHdlYnBhZ2UgdGhhdCBjYW4gYmUgbG9hZGVkIGludG8gbXVsdGlwbGUgdGFicyB0bworICAgICAg ICBjb25zdW1pbmcgc3RvcmFnZSBzcGFjZS4gT25jZSB0aGUgdGVzdCB3ZWJwYWdlcyBhcmUgbG9h ZGVkLCB5b3Ugd2lsbAorICAgICAgICBuZWVkIHRvIG1vbml0b3IgdGhlIGRhdGFiYXNlIGRpcmVj dG9yeSBhbmQgaXRzIGZpbGVzIHRvIGNvbmZpcm0gdGhhdAorICAgICAgICBncm93dGggaXMgYm91 bmRlZC4gQWxzbyB0cnkgcmVsb2FkaW5nIHRoZSB0ZXN0IGluIHRoZSB0YWJzLiBBdCBubworICAg ICAgICB0aW1lIHNob3VsZCBhbnkgZGF0YWJhc2UgZmlsZSBldmVyIGV4Y2VlZCB0aGUgcXVvdGEu CisKKyAgICAgICAgKiBNb2R1bGVzL3dlYmRhdGFiYXNlL0RhdGFiYXNlVHJhY2tlci5jcHA6Cisg ICAgICAgIChXZWJDb3JlOjpEYXRhYmFzZVRyYWNrZXI6OmdldE1heFNpemVGb3JEYXRhYmFzZSk6 CisKIDIwMTMtMDItMjIgIEFudG9pbmUgUXVpbnQgIDxncmFvdXRzQGFwcGxlLmNvbT4KIAogICAg ICAgICBFeHBvc2UgYSBsaXN0IG9mIGFsbCByZWFzb25zIHRoYXQgcXVhbGlmeSBhIFJlbmRlckxh eWVyIHRvIGJlIGNvbXBvc2l0ZWQKSW5kZXg6IFNvdXJjZS9XZWJDb3JlL01vZHVsZXMvd2ViZGF0 YWJhc2UvRGF0YWJhc2VUcmFja2VyLmNwcAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2ViQ29yZS9N b2R1bGVzL3dlYmRhdGFiYXNlL0RhdGFiYXNlVHJhY2tlci5jcHAJKHJldmlzaW9uIDE0Mzc4OCkK KysrIFNvdXJjZS9XZWJDb3JlL01vZHVsZXMvd2ViZGF0YWJhc2UvRGF0YWJhc2VUcmFja2VyLmNw cAkod29ya2luZyBjb3B5KQpAQCAtMjkwLDcgKzI5MCwxOSBAQCB1bnNpZ25lZCBsb25nIGxvbmcg RGF0YWJhc2VUcmFja2VyOjpnZXRNCiAgICAgTXV0ZXhMb2NrZXIgbG9ja0RhdGFiYXNlKG1fZGF0 YWJhc2VHdWFyZCk7CiAgICAgTG9ja2VyPE9yaWdpblF1b3RhTWFuYWdlcj4gcXVvdGFNYW5hZ2Vy TG9ja2VyKG9yaWdpblF1b3RhTWFuYWdlcigpKTsKICAgICBTZWN1cml0eU9yaWdpbiogb3JpZ2lu ID0gZGF0YWJhc2UtPnNlY3VyaXR5T3JpZ2luKCk7Ci0gICAgcmV0dXJuIHF1b3RhRm9yT3JpZ2lu Tm9Mb2NrKG9yaWdpbikgLSBvcmlnaW5RdW90YU1hbmFnZXIoKS5kaXNrVXNhZ2Uob3JpZ2luKSAr IFNRTGl0ZUZpbGVTeXN0ZW06OmdldERhdGFiYXNlRmlsZVNpemUoZGF0YWJhc2UtPmZpbGVOYW1l KCkpOworCisgICAgdW5zaWduZWQgbG9uZyBsb25nIHF1b3RhID0gcXVvdGFGb3JPcmlnaW5Ob0xv Y2sob3JpZ2luKTsKKyAgICB1bnNpZ25lZCBsb25nIGxvbmcgZGlza1VzYWdlID0gb3JpZ2luUXVv dGFNYW5hZ2VyKCkuZGlza1VzYWdlKG9yaWdpbik7CisgICAgdW5zaWduZWQgbG9uZyBsb25nIGRh dGFiYXNlRmlsZVNpemUgPSBTUUxpdGVGaWxlU3lzdGVtOjpnZXREYXRhYmFzZUZpbGVTaXplKGRh dGFiYXNlLT5maWxlTmFtZSgpKTsKKworICAgIC8vIEEgcHJldmlvdXMgZXJyb3IgbWF5IGhhdmUg YWxsb3dlZCB0aGUgb3JpZ2luIHRvIGV4Y2VlZCBpdHMgcXVvdGEsIG9yIG1heQorICAgIC8vIGhh dmUgYWxsb3dlZCB0aGlzIGRhdGFiYXNlIHRvIGV4Y2VlZCBvdXIgY2FjaGVkIGVzdGltYXRlIG9m IHRoZSBvcmlnaW4KKyAgICAvLyBkaXNrIHVzYWdlLiBEb24ndCBtdWx0aXBseSB0aGF0IGVycm9y IHRocm91Z2ggaW50ZWdlciB1bmRlcmZsb3csIG9yIHRoZQorICAgIC8vIGVmZmVjdGl2ZSBxdW90 YSB3aWxsIHBlcm1hbmVudGx5IGJlY29tZSAyXjY0LgorICAgIHVuc2lnbmVkIGxvbmcgbG9uZyBt YXhTaXplID0gcXVvdGEgLSBkaXNrVXNhZ2UgKyBkYXRhYmFzZUZpbGVTaXplOworICAgIGlmICht YXhTaXplID4gcXVvdGEpCisgICAgICAgIG1heFNpemUgPSAwOworICAgIHJldHVybiBtYXhTaXpl OwogfQogCiB2b2lkIERhdGFiYXNlVHJhY2tlcjo6ZGF0YWJhc2VDaGFuZ2VkKERhdGFiYXNlQmFj a2VuZEJhc2UqIGRhdGFiYXNlKQo= 190085 2013-02-25 10:17:12 -0800 2013-02-25 10:42:36 -0800 Patch for returning current database size instead of 0. bug-110557b.patch text/plain 2231 mark.lam SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDE0MzkzNykKKysrIFNvdXJjZS9XZWJDb3JlL0NoYW5n ZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDMwIEBACisyMDEzLTAyLTI1ICBNYXJrIExh bSAgPG1hcmsubGFtQGFwcGxlLmNvbT4KKworICAgICAgICBXaGVuIGEgcXVvdGEgYnJlYWNoIGlz IGRldGVjdGVkLCBjaGFuZ2VkIERhdGFiYXNlVHJhY2tlcjo6Z2V0TWF4U2l6ZUZvckRhdGFiYXNl KCkKKyAgICAgICAgdG8gcmV0dXJuIHRoZSBwcmV2aW91cyBkYXRhYmFzZSBzaXplIGluc3RlYWQg b2YgMC4KKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTEx MDU1Ny4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBU ZXN0aW5nIG9mIGNvbmN1cnJlbnQgbXVsdGktcHJvY2VzcyBjb25zdW1wdGlvbiBvZiBkYXRhYmFz ZSBxdW90YSBzaG93cyB0aGF0CisgICAgICAgIHJldHVybmluZyBhIHZhbHVlIG9mIDAgd2hlbiB0 aGUgcXVvdGEgaXMgZXhjZWVkZWQgc3RpbGwgYWxsb3dzIHNvbWUgZGF0YWJhc2VzCisgICAgICAg IHRvIHJ1biBhd2F5IHdpdGggdW5ib3VuZGVkIGdyb3d0aC4gSG93ZXZlciwgaWYgZ2V0TWF4U2l6 ZUZvckRhdGFiYXNlKCkgcmV0dXJucworICAgICAgICB0aGUgZXhpc3RpbmcgZGF0YWJhc2Ugc2l6 ZSwgdGhlIHVuZGVybHlpbmcgc3FsaXRlMyBkYXRhYmFzZSB3aWxsIHN1Y2Nlc3NmdWxseQorICAg ICAgICByZWplY3QgbmV3IGdyb3d0aC4KKworICAgICAgICBUaGUgdmFsdWUgcmV0dXJuZWQgYnkg RGF0YWJhc2VUcmFja2VyOjpnZXRNYXhTaXplRm9yRGF0YWJhc2UoKSBpcyB1c2VkIHRvIHNldAor ICAgICAgICB0aGUgU1FMaXRlMyBkYXRhYmFzZSBzaXplIHVzaW5nIGEgc3FsIGNvbW1hbmQgIlBS QUdNQSBtYXhfcGFnZV9jb3VudCA9IDxzaXplPiIuCisgICAgICAgIFRoZSBTUUxpdGUzIGRvY3Vt ZW50YXRpb24gb24gdGhpcyBwcmFnbWEgc2F5cywgIlRoZSBtYXhpbXVtIHBhZ2UgY291bnQgY2Fu bm90CisgICAgICAgIGJlIHJlZHVjZWQgYmVsb3cgdGhlIGN1cnJlbnQgZGF0YWJhc2Ugc2l6ZS4i CisKKyAgICAgICAgSXQgaXMgdW5kZWZpbmVkIHdoYXQgc2V0dGluZyBpdCB0byBhIHJlZHVjZWQg c2l6ZSB3aWxsIGRvLiBTbywgd2UncmUgY2hhbmdpbmcKKyAgICAgICAgZ2V0TWF4U2l6ZUZvckRh dGFiYXNlKCkgdG8gcmV0dXJuIHRoZSBleGlzdGluZyBzaXplIGluc3RlYWQuCisKKyAgICAgICAg Tm8gbmV3IHRlc3RzLgorCisgICAgICAgICogTW9kdWxlcy93ZWJkYXRhYmFzZS9EYXRhYmFzZVRy YWNrZXIuY3BwOgorICAgICAgICAoV2ViQ29yZTo6RGF0YWJhc2VUcmFja2VyOjpnZXRNYXhTaXpl Rm9yRGF0YWJhc2UpOgorCiAyMDEzLTAyLTI1ICBNYXJpdXN6IEdyemVnb3JjenlrICA8bWFyaXVz ei5nQHNhbXN1bmcuY29tPgogCiAgICAgICAgIFtFRkxdW0dUS10gTW92ZSB0ZXh0IHNlbGVjdGlv bi9mb2N1cyBub3RpZmljYXRpb24gZm9yIGExMXkgZnJvbSBndGsgdG8gYXRrIGRpcmVjdG9yeQpJ bmRleDogU291cmNlL1dlYkNvcmUvTW9kdWxlcy93ZWJkYXRhYmFzZS9EYXRhYmFzZVRyYWNrZXIu Y3BwCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT0KLS0tIFNvdXJjZS9XZWJDb3JlL01vZHVsZXMvd2ViZGF0YWJhc2UvRGF0 YWJhc2VUcmFja2VyLmNwcAkocmV2aXNpb24gMTQzODM1KQorKysgU291cmNlL1dlYkNvcmUvTW9k dWxlcy93ZWJkYXRhYmFzZS9EYXRhYmFzZVRyYWNrZXIuY3BwCSh3b3JraW5nIGNvcHkpCkBAIC0z MDEsNyArMzAxLDcgQEAgdW5zaWduZWQgbG9uZyBsb25nIERhdGFiYXNlVHJhY2tlcjo6Z2V0TQog ICAgIC8vIGVmZmVjdGl2ZSBxdW90YSB3aWxsIHBlcm1hbmVudGx5IGJlY29tZSAyXjY0LgogICAg IHVuc2lnbmVkIGxvbmcgbG9uZyBtYXhTaXplID0gcXVvdGEgLSBkaXNrVXNhZ2UgKyBkYXRhYmFz ZUZpbGVTaXplOwogICAgIGlmIChtYXhTaXplID4gcXVvdGEpCi0gICAgICAgIG1heFNpemUgPSAw OworICAgICAgICBtYXhTaXplID0gZGF0YWJhc2VGaWxlU2l6ZTsKICAgICByZXR1cm4gbWF4U2l6 ZTsKIH0KIAo=